Skeleton UI needs updating to Proton tab style
Categories
(Firefox :: Theme, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox89 | --- | fixed |
People
(Reporter: jaws, Assigned: alexical)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [proton-tabs-bar])
Attachments
(2 files, 1 obsolete file)
During startup the Skeleton UI shows the Photon tab shape. This will need to be updated to show the Proton (disconnected) tab shape before we ship Proton in Firefox 89.
Doug, is this something you can look in to?
Assignee | ||
Comment 1•4 years ago
|
||
Yup! It's already in our plans - just didn't have a bug yet. So thank you! Do you think the Proton designs are stable enough now to develop against?
Reporter | ||
Comment 2•4 years ago
|
||
Yes, they are stable enough now to develop against. Thanks!
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Updated•4 years ago
|
Reporter | ||
Updated•4 years ago
|
Assignee | ||
Comment 3•4 years ago
|
||
Assignee | ||
Comment 4•4 years ago
|
||
We specify our colors like 0xrrggbb, but CreateSolidBrush expects a
color created with the RGB macro, which actually arranges it as
0xbbggrr. This change has us use the RGB macro to give a color when
creating the brush.
Depends on D107944
Assignee | ||
Comment 5•4 years ago
|
||
This causes them to match the figma mocks, not the currently implemented
colors.
Depends on D107945
Comment 6•4 years ago
|
||
We discussed at our MR1 product experience review that we should handle skeleton UI / MR1 interdependencies as follows:
- Skeleton UI availability should not block shipping MR1
- Skeleton UI should not ship until this bug is resolved
Updated•4 years ago
|
Updated•4 years ago
|
Comment 7•4 years ago
|
||
Can I please have a status update here? Per Comment 6 Skeleton UI won't block MR1 but should not make it to Beta if this bug is not resolved.
Assignee | ||
Comment 8•4 years ago
|
||
Waiting on review from emalysz. Throwing the needinfo her way, though I know she's looking at it as we talked about it yesterday. I don't have any worries about this landing very shortly.
Comment 9•4 years ago
|
||
We'll make sure to land this patch later today before soft freeze
Comment 10•4 years ago
|
||
Comment 11•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0f69c907eba1
https://hg.mozilla.org/mozilla-central/rev/5632c59062ad
Description
•