Closed
Bug 2044354
Opened 6 days ago
Closed 4 days ago
Update the Containers colors
Categories
(Firefox :: Containers, task)
Firefox
Containers
Tracking
()
RESOLVED
FIXED
153 Branch
| Tracking | Status | |
|---|---|---|
| firefox153 | --- | fixed |
People
(Reporter: baku, Assigned: baku, NeedInfo)
References
Details
(Keywords: dev-doc-needed)
Attachments
(1 file)
Nova has a new set of colors for Containers. This bug will be moved to the Containers bugzilla components as soon as available (bug 2044350).
For this bug we need to:
- support the nova colors when nova is active
- map old containers colors to the new ones. The differences are: turquoise -> cyan and toolbar -> gray
- implement the color migration in ContextualIdentityService
- support legacy colors in the webext API
| Assignee | ||
Comment 1•6 days ago
|
||
Updated•6 days ago
|
Assignee: nobody → amarchesini
Status: NEW → ASSIGNED
| Assignee | ||
Comment 2•5 days ago
|
||
We need to update the list of colors: https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/contextualIdentities/update
Keywords: dev-doc-needed
| Assignee | ||
Updated•5 days ago
|
Component: Settings UI → Containers
Pushed by amarchesini@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/c775ae538fcc
https://hg.mozilla.org/integration/autoland/rev/80b5ebf490f9
Containers: nova color migration r=mkaply,extension-reviewers,fluent-reviewers,flod,willdurand,robwu
Comment 4•4 days ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 days ago
status-firefox153:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 153 Branch
Comment 5•9 hours ago
|
||
Richard, could you update the contextualIdentities article on MDN (and the release notes) and BCD to mention the color changes?
Note that the change of these colors is why we have a new extension API to enumerate supported colors (bug 2044712), to enable extensions to be compatible with other color changes in the future, without requiring them to hardcode the colors.
Flags: needinfo?(rbloor)
See Also: → 2044712
You need to log in
before you can comment on or make changes to this bug.
Description
•