Non-native theme button/dropdown menu are not visible with high contrast
Categories
(Core :: Widget, defect)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox88 | --- | verified |
People
(Reporter: Gabi, Assigned: emilio)
References
Details
Attachments
(4 files)
Screenshot_2021-03-10 highcontrast.png
Affected versions
- Nightly 88.0a1
Affected platforms
- Windows 10x64
Steps to reproduce
- Launch Firefox Nightly
- Enable high contrast mode (Alt+Shift+PtrScr)
- Go to test.html link
- Observe the button form control
Expected result
- Button form control is visible with high contrast mode
Actual result
- Button form control is not visible with high contrast
Regression range
Not a regression
Additional notes
- Button/dropdowns are visible when setting pref "widget.non-native-theme.enabled" = false
- https://www.groupon.com dropdown menus are not visible with high contrast
|
Reporter | |
Updated•4 years ago
|
|
|
Reporter | |
Comment 1•4 years ago
|
||
|
Assignee | |
Updated•4 years ago
|
|
|
Assignee | |
Comment 2•4 years ago
|
||
Linux can also have high contrast (and mac, if you tweak prefs, but
let's assume that doesn't happen), so no reason we shouldn't share this
code.
One related simplification while I was doing this code move is that I
managed to remove the scrollbar "border" code. Turns out that Windows
was overriding ComputeScrollbarColors so that border and track colors
were always the same, and Linux was ignoring the border anyways, so with
this we can simplfiy the implementation a bit (as the Linux scrollbar
track / corner code can be shared with Windows now).
|
|
Assignee | |
Comment 3•4 years ago
|
||
As in that case we will actually use system colors.
Depends on D107863
|
||
Comment 6•4 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/483806cf81ce
https://hg.mozilla.org/mozilla-central/rev/d465f635cd83
https://hg.mozilla.org/mozilla-central/rev/f5231f80b867
|
|
Reporter | |
Comment 7•4 years ago
|
||
Verified as fixed with Nightly 88.0a1 on Windows 10x64.
|
|
Reporter | |
Updated•4 years ago
|
Description
•