Closed
Bug 1930284
Opened 4 months ago
Closed 4 months ago
Clean up toolbox color declarations.
Categories
(Firefox :: Theme, task)
Firefox
Theme
Tracking
()
RESOLVED
FIXED
134 Branch
Tracking | Status | |
---|---|---|
firefox134 | --- | fixed |
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 files)
No description provided.
Assignee | ||
Comment 1•4 months ago
|
||
While writing bug 1928151, I realized we really don't need the separate
non-lwt versions of this. We only use them to set the general variables,
so they can just go away.
Assignee | ||
Comment 2•4 months ago
|
||
Unlike the previous patch this does change behavior, but I think it's
fine.
The reason we used the non-lwt toolbar colors for customize mode is to
guarantee full opacity, see bug 1762379 and bug 1765685.
However, we need to do the same on the sidebar, and there we chose a
more theme-friendly way of doing that, which allows this simplification.
So do that instead.
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c2fd6f284a05
Clean up toolbox color variables. r=desktop-theme-reviewers,dao
Assignee | ||
Updated•4 months ago
|
Keywords: leave-open
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b8fbcb70a97d
Also clean-up toolbar color variables. r=desktop-theme-reviewers,dao
Assignee | ||
Updated•4 months ago
|
Keywords: leave-open
https://hg.mozilla.org/mozilla-central/rev/c2fd6f284a05
https://hg.mozilla.org/mozilla-central/rev/b8fbcb70a97d
Status: NEW → RESOLVED
Closed: 4 months ago
status-firefox134:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•