Closed
Bug 1506447
Opened 6 years ago
Closed 6 years ago
scrollbar-color results in a black background on macOS Mojave
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
VERIFIED
FIXED
mozilla65
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox63 | --- | unaffected |
firefox64 | + | verified |
firefox65 | + | verified |
People
(Reporter: fvsch, Assigned: xidorn)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(4 files)
(deleted),
image/png
|
Details | |
(deleted),
text/x-phabricator-request
|
lizzard
:
approval-mozilla-beta+
|
Details |
(deleted),
text/html
|
Details | |
(deleted),
image/png
|
Details |
Using :root { scrollbar-color: ... } on macOS Mojave with Dark Mode enabled results in scrollbars showing up with a fully black background when scrolling.
This affects web content as shown in the attached screenshot.
This may effect Mojave's light mode as well
I've also noticed something similar in DevTools, in the Style Editor or the Debugger's center panel, but not sure it's the same issue.
Assignee | ||
Updated•6 years ago
|
Component: CSS Parsing and Computation → Widget: Cocoa
Assignee | ||
Updated•6 years ago
|
Flags: needinfo?(xidorn+moz)
Assignee | ||
Comment 1•6 years ago
|
||
I cannot reproduce this issue on macOS Mojave with either dark mode or light mode.
Flags: needinfo?(xidorn+moz)
Reporter | ||
Comment 2•6 years ago
|
||
Reproduced in this video with Mojave 10.14.1 in Dark Mode, latest Nightly and a fresh profile:
https://www.dropbox.com/s/2qgkp8oo3g46xlu/fx-mojave-scrollbar-color.mov?dl=1
In the first part of the video we can see strange things happening with scrollbars in devtools Style Editor and Debugger, though I'm not sure if this is related to our use of scrollbar-color or not.
In the second part you can see how using scrollbar-color affects the scrollbar's rendering when scrolling.
Assignee | ||
Comment 3•6 years ago
|
||
Thanks for the screencast, that's very helpful. I now know how to reproduce it, and I'll have a look into the issue.
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → xidorn+moz
Blocks: 1498216
status-firefox64:
--- → affected
status-firefox65:
--- → affected
tracking-firefox64:
--- → ?
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Comment 5•6 years ago
|
||
This would need uplifting. But the code should be low risk enough.
Assignee | ||
Comment 6•6 years ago
|
||
Updated•6 years ago
|
status-firefox63:
--- → unaffected
status-firefox-esr60:
--- → unaffected
tracking-firefox65:
--- → +
Keywords: regression
Pushed by mozilla@upsuper.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a6ccef7ece93
Check overlay before checking custom scrollbar style in nsNativeThemeCocoa::GetWidgetTransparency. r=spohl
Assignee | ||
Comment 8•6 years ago
|
||
Assignee | ||
Comment 9•6 years ago
|
||
Comment on attachment 9027390 [details]
Bug 1506447 - Check overlay before checking custom scrollbar style in nsNativeThemeCocoa::GetWidgetTransparency. r?spohl
[Beta/Release Uplift Approval Request]
Feature/Bug causing the regression: Bug 1498216
User impact if declined: Users on macOS Mojave may see problematic scrollbars (as shown in the attachment) when visiting a website using the new scrollbar-color property.
Is this code covered by automated tests?: No
Has the fix been verified in Nightly?: No
Needs manual test from QE?: Yes
If yes, steps to reproduce: On macOS Mojave, ensure that scrollbar is autohidden, then open the testcase attachment 9028594 [details] and scroll with touchpad. The scrollbar should not be black. This may be more reproducible with dark mode.
List of other uplifts needed: None
Risk to taking this patch: Low
Why is the change risky/not risky? (and alternatives if risky): This change only touches code path used by the new feature, which isn't widely used yet, so it's unlikely to break any existing stuff.
String changes made/needed:
Attachment #9027390 -
Flags: approval-mozilla-beta?
Comment 10•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Flags: qe-verify+
Verified the fix on the Nightly 12-01 build.
Comment on attachment 9027390 [details]
Bug 1506447 - Check overlay before checking custom scrollbar style in nsNativeThemeCocoa::GetWidgetTransparency. r?spohl
Fix for new macOS scrollbar issue, verified in nightly, let's uplift.
[Triage Comment]
Attachment #9027390 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 13•6 years ago
|
||
bugherder uplift |
Comment 14•6 years ago
|
||
I verified this on Mac OS X 10.14 on FF Nightly 65.0a1(2018-12-02) and I can't reproduce the issue. I will verify it on Beta after the uplift.
Status: RESOLVED → VERIFIED
Comment 15•6 years ago
|
||
I also tested this on iMac OS X 10.14 with FF Nightly 65.0a1(2018-12-02) and I think there is a problem, please see the attached print screen with the actual result on iMac. Please note that on MacBook this issue is not displayed on the same Firefox version.
Xidorn can you please take a look? Thanks
Flags: needinfo?(xidorn+moz)
Assignee | ||
Comment 16•6 years ago
|
||
Scrollbar with blue track and cyan thumb is expected when overlay scrollbar is not used in the given test case.
Flags: needinfo?(xidorn+moz)
Comment 17•6 years ago
|
||
I tested this on MacBook 10.14 with FF Beta 64.0b14 and I can confirm the fact that this issue is fixed.
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•