Closed Bug 1486740 Opened 6 years ago Closed 6 years ago

[MAC] Text blurry after changing to dark theme

Categories

(Firefox :: New Tab Page, defect, P3)

All
macOS
defect

Tracking

()

RESOLVED DUPLICATE of bug 1482534
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- unaffected
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- wontfix
firefox64 --- fix-optional

People

(Reporter: cfogel, Unassigned)

References

Details

(Keywords: regression, uiwanted)

Attachments

(1 file)

[Affected versions]:
- 63.0b1_DevED, 62.0b19, 63.0a1 (2018-08-23),61.0.2, 

[Affected platforms]:
- macOS 10.13, macOS 10.12

[Steps to reproduce]:
0. Firefox is set to Light/Default/Any other theme.
 
1. Launch Firefox;
2. Open the newtab page;
3. Click on the Hamburger menu, click on the Customize option;
4. Click on the Theme picker, choose the Dark theme;
5. Click on the Done button;
6. Press F12 on the keyboard(opening devTools);
7. Click on the Inspector tab for the devTools.

[Expected result]:
- page content is properly rendered;

[Actual result]:
- changing to dark theme text on new-tab is blurry;
- opening the DevTools-inspector clears the blur effect.

[Regression range]:
- while checking for regression I considered as issue the fact that the styling changes after toggling the devTools inspector on; Current nightly has the text crisp/thin/readable;

- mozregression result:
Last good revision: 1344ca26ef0aec57cb0177f7089287b263c7a52b
First bad revision: 9896401414cb3c7cb5f1118b307b0267212c8aad

Pushlog URL:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=1344ca26ef0aec57cb0177f7089287b263c7a52b&tochange=9896401414cb3c7cb5f1118b307b0267212c8aad

[Additional notes]:
- recording with the issue: http://g.recordit.co/XAl1OTAiNl.gif
- noticeable on text from topSites, Highlights as well, in some cases even the text from the topSites tiles was affected;
Attached the recording with the issue.
Component: Theme → Graphics: Text
Product: Firefox → Core
Component: Graphics: Text → Activity Streams: Newtab
Product: Core → Firefox
Keywords: uiwanted
Might be related or the same as bug 1482534 ? -moz-osx-font-smoothing: grayscale

cfogel, does unchecking/disabling that style with devtools fix the blurriness?

The style shows up as

[lwt-newtab-brighttext] {
  -moz-osx-font-smoothing: grayscale; }
See Also: → 1482534
(In reply to Ed Lee :Mardak from comment #2)

> does unchecking/disabling that style with devtools fix the blurriness?


It doesn't seem to be it. 
 
Have tried with creating a separate html file and applying just that property to it, but it's not having the same effect.
I even tried copying the newtab page content to a local file but the text was properly displayed.

Could it by a combination of bug(1482534) and something else causing it?
Why are you using -moz-osx-font-smoothing: grayscale in the first place?
Priority: -- → P3
Too late to fix in 63. We could still take a patch for 65 and potentially for 64.
(In reply to Ed Lee :Mardak from comment #5)
> It was added in https://github.com/mozilla/activity-stream/pull/4089 to make
> text sharper:
> 
> https://user-images.githubusercontent.com/1455535/38701664-7a625340-3e6c-
> 11e8-9402-8e53527ce5b0.png

Clearly this isn't always working as intended. Can we just revert this for now?
Flags: needinfo?(edilee)
Aaron, is it better to just go back to the "before" https://user-images.githubusercontent.com/1455535/38701664-7a625340-3e6c-11e8-9402-8e53527ce5b0.png
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(edilee) → needinfo?(abenson)
Resolution: --- → DUPLICATE
No, the before is not the ideal. The after looks more correct (less blurry).
Flags: needinfo?(abenson)
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: