1136231 - Unify Reader View type options to be Sans-Serif on Left, Serif on Right

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Reader View, defect)

Description

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

As per our IRC convo, just defaulting to the left (Sans-serif).

Consistent across Android, iOS, and Desktop.

Comment 1

[Michael Maslaney [:mmaslaney] ([e-mail])]

Update from Desktop: http://invis.io/XT21LURF8

Comment 2

[:Margaret Leibovic]

Attachment: MozReview Request: bz://1136231/margaret

/r/4251 - Bug 1136231 - Unify Reader View type options to be Sans-Serif on Left, Serif on Right. r=bnicholson

Pull down this commit:

hg pull review -r efc517aba91c072b25a52b20ab3c717b763c040c

Comment 3

[Brian Nicholson (:bnicholson)]

Comment on attachment 8568751 [details]
MozReview Request: bz://1136231/margaret

https://reviewboard.mozilla.org/r/4249/#review3419

Ship It!

Comment 4

[:Margaret Leibovic]

https://hg.mozilla.org/integration/fx-team/rev/8f6fd5d64969

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/8f6fd5d64969

Comment 6

[Teodora Vermesan (:TeoVermesan)]

Tapping on the "Layout options" (Aa) from the toolbar, "Sans-serif" and "Serif" fonts are displayed.
Verified as fixed on;
Device: Alcatel One Touch (Android 4.1.2)
Build: Firefox for Android 39.0a1 (2015-03-01)

Comment 7

[:Margaret Leibovic]

https://hg.mozilla.org/releases/mozilla-aurora/rev/fa8645257f11

Comment 8

[Teodora Vermesan (:TeoVermesan)]

Tapping on the "Layout options" (Aa) from the reader mode toolbar, "Sans-serif" is displayed left and "Serif" is displayed right.
Verified as fixed on;
Device: Alcatel One Touch (Android 4.1.2)
Build: Firefox for Android 38.0a2 (2015-03-06)

Comment 10

[:Margaret Leibovic]

Attachment: MozReview Request: Bug 1136231 - Unify Reader View type options to be Sans-Serif on Left, Serif on Right. r=bnicholson