1102444 - Text in Help subview is using grayscale anti-aliasing while animating

Status: REOPENED

(Core :: Graphics: Text, defect)

Description

[Elbart]

Attachment: help grayscale.png

Nightly 2014-11-20

After changing from mainpanel to the help-subview, the text noticeably changes from grayscale antialiasing to cleartype.

m-c:
Last good revision: b85c260821ab (2014-10-03)
First bad revision: 229801d17f7e (2014-10-04)
Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b85c260821ab&tochange=229801d17f7e

m-i:
Last good revision: 60e90dd7f64f
First bad revision: 7a45d556f280
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=60e90dd7f64f&tochange=7a45d556f280

fx-team:
Last good revision: 849c7d93fba1
First bad revision: cb6a72e1fd95
Pushlog:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=849c7d93fba1&tochange=cb6a72e1fd95

cb6a72e1fd95	Richard Marti — Bug 1008603 - Define panel-subviews background-color in platform files. r=dao

Comment 1

[Elbart]

Attachment: help grayscale.mp4

Comment 2

[Dão Gottwald [:dao]]

(In reply to Elbart from comment #0)
> Created attachment 8526235 [details]
> help grayscale.png
> 
> Nightly 2014-11-20
> 
> After changing from mainpanel to the help-subview, the text noticeably
> changes from grayscale antialiasing to cleartype.
> 
> m-c:
> Last good revision: b85c260821ab (2014-10-03)

What does "good" mean here? That the text used grayscale AA all the time or that it used cleartype throughout?

Comment 3

[:Gijs (he/him)]

(In reply to Dão Gottwald [:dao] from comment #2)
> (In reply to Elbart from comment #0)
> > Created attachment 8526235 [details]
> > help grayscale.png
> > 
> > Nightly 2014-11-20
> > 
> > After changing from mainpanel to the help-subview, the text noticeably
> > changes from grayscale antialiasing to cleartype.
> > 
> > m-c:
> > Last good revision: b85c260821ab (2014-10-03)
> 
> What does "good" mean here? That the text used grayscale AA all the time or
> that it used cleartype throughout?

I'm fairly sure this used to be grayscale throughout, because the background wasn't opaque before the cset identified, which would have triggered grayscale AA.

It's possible we can make it use cleartype throughout assuming animations don't force it off by ensuring the background is opaque while animating as well (assuming that doesn't look too ugly of itself). If that is the case already, I'm not sure we can do anything here besides waiting for graphics/layout to allow this to be the case on animating backgrounds.

Comment 4

[Dão Gottwald [:dao]]

(In reply to :Gijs Kruitbosch from comment #3)
> (In reply to Dão Gottwald [:dao] from comment #2)
> > (In reply to Elbart from comment #0)
> > > Created attachment 8526235 [details]
> > > help grayscale.png
> > > 
> > > Nightly 2014-11-20
> > > 
> > > After changing from mainpanel to the help-subview, the text noticeably
> > > changes from grayscale antialiasing to cleartype.
> > > 
> > > m-c:
> > > Last good revision: b85c260821ab (2014-10-03)
> > 
> > What does "good" mean here? That the text used grayscale AA all the time or
> > that it used cleartype throughout?
> 
> I'm fairly sure this used to be grayscale throughout, because the background
> wasn't opaque before the cset identified, which would have triggered
> grayscale AA.

That's my understanding too, which then means that bug 1008603 didn't regress anything but was a step in the right direction, as subpixel AA is preferable over grayscale.

Comment 5

[Elbart]

(In reply to Dão Gottwald [:dao] from comment #4)
> That's my understanding too, which then means that bug 1008603 didn't
> regress anything but was a step in the right direction, as subpixel AA is
> preferable over grayscale.

Right, as Gijs Kruitbosch said, before it was grayscale-only.
I filed this just because there's now a noticeable change, depending on the Cleartype-settings of the user.

Comment 6

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

Can you still reproduce the issue in latest stable Firefox version or in latest Nightly?

Comment 7

[Elbart]

Yes.

Comment 8

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

Please attach as attachment here your about:support (use "copy text to clipboard" button) from Nightly.

Comment 9

[Elbart]

It's not depending on gfx-card or driver-version.

In the latest nightlies all text-captions in the hamburglar-menu are flickering during that animation, in both the normal pane and the help-pane.

Comment 10

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

I wasn't talking about dependency, I was talking about complete information from about:support from Nightly. Could you post it?

(In reply to Elbart from comment #9)
> In the latest nightlies all text-captions in the hamburglar-menu are
> flickering during that animation, in both the normal pane and the help-pane.
It's other bug (probably bug #1306670), not this one.

Comment 11

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

I'm asking because it could be fixed in Nightly which contains fixes for bug #1007702 and bug #1303570,
about:support is needed for checking what preferences are modified and backended is used where.

Comment 12

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

It should be fixed starting from Mozilla Firefox 52 per:
- Skia being enabled by default(Windows - bug #1007702, Linux - bug #1278957, Mac - bug #1207332)
- bug #1303570 (leftovers will be tracked in bug #1307833).

Duping this bug to bug #1307833 to later easier tracking, instead of incomplete or worksforme, because of no reporter reply.

Comment 13

[Dão Gottwald [:dao]]

Using Skia doesn't magically make us use sub-pixel AA where we were using grayscale AA. Please test this before concluding that it's fixed.

Comment 14

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

Maybe you missed, but I also wrote about bug #1303570.

Comment 15

[fireattack [:fireattack]]

It does fixed? Or did I miss something?

Tested in 53.0a1 (2016-11-18) (32-bit)

Comment 16

[fireattack [:fireattack]]

Attachment: tested as fixed in 11-28 nightly build

Comment 17

[fireattack [:fireattack]]

is*

Comment 18

[:Gijs (he/him)]

Dão, are you still reproducing issues here or should this be marked fixed/duped after all?

Comment 19

[Dão Gottwald [:dao]]

I can still reproduce this with the latest Nightly on Windows 10.

Comment 20

[fireattack [:fireattack]]

Oh, never mind, it is just my testing error, I didn't take the screenshot fast enough.

Please help delete my attachment to reduce future confusion.