Closed Bug 1396637 Opened 7 years ago Closed 6 years ago

Text shift on hover on Mozilla Gfx Team Blog with WebRender enabled

Categories

(Core :: Graphics: WebRender, defect, P1)

57 Branch
defect

Tracking

()

VERIFIED FIXED
mozilla60
Tracking Status
firefox56 --- unaffected
firefox57 --- unaffected
firefox58 --- unaffected
firefox59 --- unaffected
firefox60 --- unaffected

People

(Reporter: yoasif, Assigned: lsalzman)

References

(Blocks 1 open bug, )

Details

(Keywords: correctness, nightly-community, Whiteboard: [wr-reserve] [gfx-noted])

Attachments

(2 files, 1 obsolete file)

Attached image jumpy on hover.gif (obsolete) —
User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20170904100131

Steps to reproduce:

Enable WebRender with steps:


    turn off layers.async-pan-zoom.enabled
    turn on gfx.webrender.enabled
    turn on gfx.webrendest.enabled
    turn on gfx.webrender.layers-free
    add and turn on gfx.webrender.blob-images
    if you are on Linux, turn on layers.acceleration.force-enabled

Naviagate to https://mozillagfx.wordpress.com/

Hover over links for any of:

WebRender newsletter #1
WebRender newsletter #2
WebRender newsletter #3


Actual results:

The link text jumps up and to the left, then snaps back into place with the on hover color. 


Expected results:

Text should not have jumped, color should have just changed. 

See screengrab.
Component: Untriaged → Graphics: WebRender
Product: Firefox → Core
On mac, blob-images and APZ-off doesn't seem to matter.

Probably my fault somehow, not sure why yet.
Ah, for some reason those have a CSS transition, which is what causes it. We're not very good at that kind of thing in webrender right now. 

Kats, any idea if we've seen this one before?
Flags: needinfo?(bugmail)
Priority: P3 → P2
Whiteboard: [gfx-noted] → [wr-mvp] [gfx-noted]
I don't recall any exising bug on file that would cover this, no.
Flags: needinfo?(bugmail)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Nightly 58 x64 20171020100426 de_DE @ Debian Testing (KDE, Radeon RX480).
main profile: layers force accel + webrender + webrendest + blob-image + gpu process + omtp

WFM

(In reply to Asif Youssuff from comment #0)
Can you confirm this?
Attached video moving-text.mp4
It is a lot better now, it jumps a lot less. It's more of a small shift. This doesn't occur in a fresh nightly profile, however, so it continues to be unexpected. 

Updated video attached.
Attachment #8904317 - Attachment is obsolete: true
Summary: Jumpy text on hover on Mozilla Gfx Team Blog with WebRender enabled → Text shift on hover on Mozilla Gfx Team Blog with WebRender enabled
Priority: P2 → P3
Whiteboard: [wr-mvp] [gfx-noted] → [wr-reserve] [gfx-noted]
We are hitting essentially the same problem we are with Skia - that subpixel positioning is causing us to not rasterize glyphs where Cairo metrics are putting them.

So we need the same workaround for now, just to disable subpixel positioning here until such time as we punt Cairo. This should fix the epidemic of text shifting bugs we are seeing.
Assignee: nobody → lsalzman
Status: NEW → ASSIGNED
Attachment #8949359 - Flags: review?(jmuizelaar)
Attachment #8949359 - Flags: review?(jmuizelaar) → review+
Pushed by lsalzman@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/94d10cf318b8
disable WR subpixel positioning with FreeType. r=jrmuizel
Blocks: 1436331
Blocks: 1435387
https://hg.mozilla.org/mozilla-central/rev/94d10cf318b8
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
(In reply to Asif Youssuff from comment #5)
> Created attachment 8920741 [details]
> moving-text.mp4
> 
> It is a lot better now, it jumps a lot less. It's more of a small shift. This doesn't occur in a fresh nightly profile, however, so it continues to be unexpected. 

Is it fixed?
Flags: needinfo?(yoasif)
See Also: → 1412127
It is now fixed.
Flags: needinfo?(yoasif)
No longer blocks: 1435387
No longer blocks: 1436331
Depends on: 1458921
See Also: → 1508163
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: