Closed Bug 1930284 Opened 4 months ago Closed 4 months ago

Clean up toolbox color declarations.

Categories

(Firefox :: Theme, task)

task

Tracking

()

RESOLVED FIXED
134 Branch
Tracking Status
firefox134 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(2 files)

No description provided.

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.

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
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
Keywords: leave-open
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 134 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: