Closed Bug 1760317 Opened 3 years ago Closed 3 years ago

color_scheme of the theme does not control the context menu

Categories

(Firefox :: Theme, defect)

Firefox 100
defect

Tracking

()

VERIFIED FIXED
100 Branch
Tracking Status
firefox100 --- verified

People

(Reporter: anon4ymity, Assigned: emilio)

References

Details

Attachments

(2 files)

Attached image context menu

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:98.0) Gecko/20100101 Firefox/98.0

Steps to reproduce:

Windows 10 64-Bit (21H2, Build 19044.1586)
Firefox Nightly 100a1 (2022-03-18)

Setting:
Website appearance: Nightly theme

Theme:
https://addons.mozilla.org/de/firefox/addon/tiger-eyes-turquoise/

Actual results:

The context menu and the menu of the menu bar do not follow the color_scheme of the theme. They are always dark.

I can see that windows (content, about pages, library) follow the color scheme. I can make them light or dark with the theme.

Expected results:

The context menu and menu bar are also in the light theme.

The Bugbug bot thinks this bug should belong to the 'Firefox::Theme' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Component: Untriaged → Theme
Depends on: 1750932
Flags: needinfo?(emilio)

It's both simpler and more correct.

It prevents themes that declare themselves as light but have
bright text from getting dark menus.

This behavior matches other platforms.

Also, simplify the selectors, there's no need to use :host anymore
now that we use the same code path for <select> in the parent
process (bug 1744009).

Double-checked that this doesn't regress bug 1701493 nor
bug 1702626.

Assignee: nobody → emilio
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(emilio)
Pushed by ealvarez@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e3c4aacf5575 Use prefers-color-scheme to determine windows menu color scheme. r=Gijs
Depends on: 1760342
Pushed by abutkovits@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/dd2628f88452 Fix browser_selectpopup_colors to not read the colors too early. CLOSED TREE
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
Blocks: 1760342
No longer depends on: 1760342

Reproduced the issue without the fix on Win10 using build 100.0a1(20220318212951).
Verified as fixed on Win10/Mac 10.13/ Ubuntu 20.4 using build 100.0b5 (20220412185818).

Also noticed that using this theme, Customize page is transparent as on bug 1762379 (issue reproduce before this fix).

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: