Closed
Bug 1483214
Opened 6 years ago
Closed 6 years ago
Black shadow is displayed behind the text in about:welcome page when Dark theme is enabled
Categories
(Core :: Graphics: Text, defect)
Tracking
()
RESOLVED
DUPLICATE
of bug 1462672
Tracking | Status | |
---|---|---|
firefox61 | --- | unaffected |
firefox62 | --- | wontfix |
firefox63 | --- | fixed |
People
(Reporter: Gabi, Unassigned)
References
Details
(Keywords: regression)
Attachments
(1 file)
239.10 KB,
image/png
|
Details |
[Affected versions]: - Beta 62.0b17 [Affected Platforms]: - macOS 12.6 - macOS 10.11.5 [Steps to reproduce]: 1. Open Fx with a clean profile 2. When about:welcome page is displayed go to Customize 3. Go to Themes 4. Change theme to Dark 5. Observe the about:welcome page [Expected results]: about:welcome page should be displayed properly, no UI issues should occurr [Actual results]: Black shadow is displayed behind the strings in about:welcome page when Dark theme is enabled [Notes]: - Refreshing the about:welcome page fixes the issue - Attached png of the issue - Not reproducible under Windows, Ubuntu or MacOS 10.14
Comment 1•6 years ago
|
||
(FYI: Firefox::General should only be used for bugs known not to have a better component, otherwise filing in Firefox :: Untriaged will have someone move them to the right spot.)
Component: General → Theme
Comment 2•6 years ago
|
||
Hmm, the dark theme really shouldn't affect this page at all, right?
Flags: needinfo?(ntim.bugs)
Comment 3•6 years ago
|
||
Ah, this page is basically about:home with some stuff added.
Component: Theme → Activity Streams: Newtab
Flags: needinfo?(ntim.bugs)
Comment 5•6 years ago
|
||
This can only be reproduced on a new profile, and only for a small while. The font eventually re-adjusts itself. When I go to inspect the font in the debugger, this causes a repaint and it automatically changes to appear as expected. For about:welcome's purposes, I'm not worried about this, it seems like a small edge case. Considering that on the first run the user will have Firefox set to it's default theme. However, this seems like a graphics bug to me and something they might be interested in pursuing.
Flags: needinfo?(ewright) → needinfo?(jmuizelaar)
Comment 6•6 years ago
|
||
This looks like a subpixel aa bug. Thoughts Lee?
Component: Activity Streams: Newtab → Graphics: Text
Flags: needinfo?(jmuizelaar) → needinfo?(lsalzman)
Product: Firefox → Core
Comment 7•6 years ago
|
||
This is listed as 61 and 63 being unaffected, which would imply there was a regression and that the regression was fixed by something. If so, could you please use mozregression to narrow down which revision caused it, and which revision fixed it? See https://mozilla.github.io/mozregression/quickstart.html
Flags: needinfo?(lsalzman) → needinfo?(gasofie)
Comment 8•6 years ago
|
||
This is most likely a layers issue. It is either possibly related to bug 1493075 or bug 1473041.
Comment 9•6 years ago
|
||
Gabi can you get a regression window?
Comment 10•6 years ago
|
||
This was fixed in 63 by bug 1462672. 62 has already been released, and that bug is not upliftable to a point release, so I think there's nothing we can do here. I found the fix range using $ mozregression --find-fix --bad 20180625100047 with the following STR: 1. Type about:welcome in the URL bar and press enter. 2. Open the hamburger menu and choose Customize... 3. Click on the Themes button and move the mouse upwards. 3a. If this causes the browser layout to be messed up: 3a1. Click in the gray space outside to close the menu. 3a2. Resize the browser window slightly (make it just a bit larger) by dragging the window's bottom right corner. 3a3. Switch tabs to about:welcome. 3a4. Switch tabs back to custimization. 3a5. Open the Themes menu again. 4. Select the Dark Theme. 5. Switch back to about:welcome. Fix range: https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=85f1c6e5ddac7792b327c05996f8bf27ed728d72&tochange=92bd3b2178623a036ea91990aa5570e039890ff4 Since bug 1462672 added an alternative code path for rendering this case, it's possible that this bug is still present in the old code path. It would need more investigation to find out whether that's the case.
Updated•6 years ago
|
Flags: needinfo?(gasofie)
Comment 11•6 years ago
|
||
61 does not have about:welcome. I think that's why it's "unaffected". With the same test page, I'd expect it to show this bug.
Updated•6 years ago
|
Keywords: regression
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•