1921185 - [Reader View] Add an option for justified text alignment

Status: RESOLVED FIXED

(Toolkit :: Reader Mode, enhancement)

Description

[Eugene Popov]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:130.0) Gecko/20100101 Firefox/130.0

Steps to reproduce:

Currently, Reader View has three options for text alignment: start, end, and center. It would be great if there was also an option for justified text alignment.

Comment 1

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

The Bugbug bot thinks this bug should belong to the 'Toolkit::Reader Mode' component, and is moving the bug to that component. Please correct in case you think the bot is wrong.

Comment 2

[Irene Ni [:ini]]

Attachment: Bug 1921185 - Add an option for justified text alignment. r=#reader-mode-reviewers

Comment 3

[Irene Ni [:ini]]

Gave this enhancement a try as this ticket and Bug 1913738 are both requesting it. A couple notes:

• May need to confirm with design team if align-justify-20.svg is encoded correctly, as I made it in Figma.
• I made the RTL behavior so that the justify button remains as the last in the row.
• Deviated from the "Align text {position}" labeling convention, as "Justify text" seemed more clear to me.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c2ee3fd68fcd
Add an option for justified text alignment. r=reader-mode-reviewers,fluent-reviewers,desktop-theme-reviewers,bolsson,Gijs

Comment 5

[Cosmin Sabou [:CosminS]]

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