1642166 - Vertically align the Reader control hover labels

Status: RESOLVED FIXED

(Toolkit :: Reader Mode, defect, P3)

Description

[Itiel]

Attachment: Screenshot

They should be 2px higher.

Comment 1

[Itiel]

Attachment: Bug 1642166 - Vertically align the Reader control hover labels r?gijs

Also cleanup the CSS a bit

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bef16d556f09
Vertically align the Reader control hover labels r=Gijs

Comment 3

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/bef16d556f09

Comment 4

[:Gijs (he/him)]

Comment on attachment 9152991 [details]
Bug 1642166 - Vertically align the Reader control hover labels r?gijs

Beta/Release Uplift Approval Request

• User impact if declined: hover labels in reader mode look misaligned on non-en-US
• Is this code covered by automated tests?: No
• Has the fix been verified in Nightly?: No
• Needs manual test from QE?: No
• If yes, steps to reproduce:
• List of other uplifts needed: n/a
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): This is a very small CSS-only polish patch. It's not risky.
• String changes made/needed: None

Comment 5

[Julien Cristau [:jcristau]]

Comment on attachment 9152991 [details]
Bug 1642166 - Vertically align the Reader control hover labels r?gijs

css fix for reader mode, approved for 78.0b5

Comment 6

[Julien Cristau [:jcristau]]

https://hg.mozilla.org/releases/mozilla-beta/rev/17b32bf99b46