995096 - New Sync page: title is broken for RTL languages, show unneeded horizontal scrollbar

Status: VERIFIED FIXED

(www.mozilla.org :: Pages & Content, defect)

Description

[Francesco Lodolo [:flod]]

https://www-dev.allizom.org/he/firefox/sync/

Title keeps the structure of LTR, Sync logo should move to the left. 

Looking into it.

Comment 1

[Francesco Lodolo [:flod]]

Assigning to Théo, so he can get started with Bedrock :-)

Comment 2

[Théo Chevalier [:tchevalier]]

Attachment: PR

This fixes the Sync page, (layout + scroll bar).
It also fixes Tabzilla on many pages like /he/firefox/channel/ where Tabzilla button should be on the left.

Comment 3

[Théo Chevalier [:tchevalier]]

Comment on attachment 8405630 [details] [review]
PR

Asking Jon for review since Steven is busy

Comment 4

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/bcde6f04404ccd2ea242a7d244602fb9ed534277
[Bug 995096] New Sync page: title is broken for RTL languages

https://github.com/mozilla/bedrock/commit/c691ebbfbf3870744df7f1f326d0b5a93e658530
Merge pull request #1877 from TheoChevalier/995096-sync-rtl-fix

[Bug 995096] - New Sync page: title is broken for RTL languages, show unneeded horizontal scrollbar

Comment 5

[raymond [:retornam] (inactive)]

fixed on dev and stage https://www-dev.allizom.org/he/firefox/sync/

Comment 6

[Francesco Lodolo [:flod]]

Verified on prod
https://www.mozilla.org/ar/firefox/sync/
https://www.mozilla.org/he/firefox/sync/