Closed
Bug 1930284
Opened 1 year ago
Closed 1 year 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•1 year 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•1 year 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•1 year 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•1 year ago
|
Keywords: leave-open
https://hg.mozilla.org/mozilla-central/rev/c2fd6f284a05
https://hg.mozilla.org/mozilla-central/rev/b8fbcb70a97d
Status: NEW → RESOLVED
Closed: 1 year 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
•