1366240 - Grayscale antialiasing instead of subpixel antialiasing is used in address bar after landing patches from bug #653670

Status: RESOLVED INACTIVE

(Core :: Graphics: Text, defect, P3)

Description

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

[Tracking Requested - why for this release]: Regression


STR.
1. Open Firefox
2. and see address bar font text rendering


I suspect it's regression due to some recent Photon address bar changes, which I'm yet to found in detail which one is the main cause.

Comment 1

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

Attachment: Firefox 54.png

Comment 2

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

Attachment: Firefox 55.png

Comment 3

[Dão Gottwald [:dao]]

This is a regression from bug 653670. Bug 1364387 will fix this except when the location bar is overflowing.

Comment 4

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

Thank you very much Dão Gottwald [::dao] for finding specific bug which caused this issue.

Comment 5

[Julien Cristau [:jcristau]]

Doesn't seem worth tracking for 55 release.

Comment 6

[Liz Henry (:lizzard) (relman/hg->git project)]

Sounds like this is mostly fixed in 55. We could close this issue and file a new bug for the specific case mentioned in comment 3 (maybe make it a good first bug?).

Comment 7

[Dão Gottwald [:dao]]

(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #6)
> Sounds like this is mostly fixed in 55. We could close this issue and file a
> new bug for the specific case mentioned in comment 3 (maybe make it a good
> first bug?).

Bug 1366405 should help further. The rest isn't really fixable, let alone as a good first bug.

Comment 8

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

There is always the option to back out the offending patch.

Mozilla should not add the features with not fixed regressions, especially the ones which causes the bad text font rendering and the eye strain because of it.

Comment 9

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

[Tracking Requested - why for this release]: Rerequesting tracking, as fixed bug #1366405 unfortunately didn't make any difference to this issue (at least in my case).

Comment 10

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

Ups, it's not fixed yet, I was seeing wrong bug.

Comment 11

[Liz Henry (:lizzard) (relman/hg->git project)]

I get this is still an issue and is annoying on Win 7 for some folks, but I don't think changing the flags on all the related bugs at once is going to make it more of a priority. 

Happy to take a patch in Nightly if anyone can work on this, but it isn't a release blocker.

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Too late to fix in 63. We could still take a patch for 65 and potentially for 64.

Comment 13

[Liz Henry (:lizzard) (relman/hg->git project)]

Another one that is just going to be backlog. We would take a patch but don't need to keep triaging it.

Comment 14

[Neha Kochar [:neha]]

Bulk change for all regression bugs with status-firefox67 as 'fix-optional' to be marked 'affected' for status-firefox68.

Comment 15

[Florencia Di Ciocco, NI to vbalducci]

Hi,

I get the impression that we can close this bug. Have you had this issue lately?

Regards, Flor.

Comment 16

[Virtual_ManPL [:Virtual] 🇵🇱 - (please needinfo? me - so I will see your comment/reply/question/etc.)]

Unfortunately, I can still reproduce this issue in Mozilla Firefox Nightly 87.0a1 (2021-02-04).

Comment 17

[BugBot [:suhaib / :marco/ :calixte]]

In the process of migrating remaining bugs to the new severity system, the severity for this bug cannot be automatically determined. Please retriage this bug using the new severity system.

Comment 18

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:lsalzman, could you have a look please?

For more information, please visit BugBot documentation.