Closed Bug 1358089 Opened 4 years ago Closed 4 years ago

[RTL] broken UI layout after last update


(Firefox for Android Graveyard :: Theme and Visual Design, defect)

53 Branch
Not set


(fennec53+, firefox-esr52 unaffected, firefox53+ verified, firefox54+ fixed, firefox55+ fixed)

Firefox 55
Tracking Status
fennec 53+ ---
firefox-esr52 --- unaffected
firefox53 + verified
firefox54 + fixed
firefox55 + fixed


(Reporter: tomer, Assigned: maliu)




(Keywords: regression)


(2 files)

Please note that I can't reproduce this on my devices, I am just passing user feedback. 

A user provided feedback from our community page on Facebook about regression since last update (Fennec 53):

The URL bar layout is misaligned once he click on it. See attached screenshot.

Device: OnePlus 3T, Android 7.1.1, official stock ROM, Hebrew UI.
Browser: Firefox 53 (Release channel), switching languages between Hebrew and English has no effect.

Bug is visible only when Android system UI is set the Hebrew. Bug does not reproduce on English UI. This bug is a regression since previous version, as this is the first with RTL UI enabled.
Attached image Screenshot
tracking-fennec: --- → ?
[Tracking Requested - why for this release]:
This is a regression that impacts RTL users.
53 is the release we claim the RTL support so I'm nominating this fix to be uplifted.
tracking-fennec: ? → 53+
Attachment #8861300 - Flags: review?(s.kaspari)
Leaving this for ahunt to review.
Assignee: nobody → max
It's misaligned but maybe still usable. 
Please request uplift to mozilla-release once the patch land on m-c and we can verify the fix. 
I'm not sure this will make the 53.0.2 dot release, which I'm hoping to build today.
Comment on attachment 8861300 [details]
Bug 1358089 - [RTL] Separate xml drawable into v17 folder,
Attachment #8861300 - Flags: review?(ahunt) → review+
Pushed by
[RTL] Separate xml drawable into v17 folder, r=ahunt
Comment on attachment 8861300 [details]
Bug 1358089 - [RTL] Separate xml drawable into v17 folder,

Approval Request Comment
[Feature/Bug causing the regression]: RTL Bug 1350661
[User impact if declined]: Fennec UI in LTR(English) with system locale in RTL(Hebrew) will see abnormal UI when editing in UrlBar 
[Is this code covered by automated tests?]: No
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: Set android system locale to RTL(Hebrew) and specify fennec locale to LTR(English), click on Urlbar to edit and observe.
[List of other uplifts needed for the feature/fix]: N/A
[Is the change risky?]: No
[Why is the change risky/not risky?]: Manually verified on multiple Android version.
[String changes made/needed]: No
Attachment #8861300 - Flags: approval-mozilla-release?
Attachment #8861300 - Flags: approval-mozilla-beta?
Attachment #8861300 - Flags: approval-mozilla-aurora?
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Hi Ioana, 
could you help find someone to verify if this issue was fixed as expected on a latest Nightly build? Thanks!
Flags: qe-verify+
Flags: needinfo?(ioana.chiorean)
Hi Mihai,
Can you help verify this?
Flags: needinfo?(mihai.ninu)
According to the user that reported this in the first place on our community page on Facebook, the issue is now fixed on latest Nightly.
Comment on attachment 8861300 [details]
Bug 1358089 - [RTL] Separate xml drawable into v17 folder,

Fix an RTL regression. Beta54+. Should be in 54 beta 4.
Attachment #8861300 - Flags: approval-mozilla-beta?
Attachment #8861300 - Flags: approval-mozilla-beta+
Attachment #8861300 - Flags: approval-mozilla-aurora?
Attachment #8861300 - Flags: approval-mozilla-aurora-
Comment on attachment 8861300 [details]
Bug 1358089 - [RTL] Separate xml drawable into v17 folder,

Verified fix, let's uplift it to m-r for the 53.0.2 dot release.
Attachment #8861300 - Flags: approval-mozilla-release? → approval-mozilla-release+
Device: Nexus 6 (Android 7.0)
Build 53.0.2
Duplicate of this bug: 1361498
As this was verified by Bogdan, will take out the NI
Flags: needinfo?(mihai.ninu)
Flags: needinfo?(ioana.chiorean)
Flags: qe-verify+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.