[Bug][RTL] Potentially don't set automatic direction based on the text directionality, for text across the UI
Categories
(Fenix :: General, defect)
Tracking
(firefox113 wontfix, firefox114 wontfix, firefox115 wontfix, firefox116 wontfix, firefox117 verified, firefox118 verified)
People
(Reporter: gl, Assigned: mavduevskiy)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fxdroid])
Attachments
(10 files)
58 bytes,
text/x-github-pull-request
|
Details | Review | |
32.81 KB,
image/jpeg
|
Details | |
40.64 KB,
image/jpeg
|
Details | |
29.92 KB,
image/jpeg
|
Details | |
59 bytes,
text/x-github-pull-request
|
Details | Review | |
59 bytes,
text/x-github-pull-request
|
Details | Review | |
59 bytes,
text/x-github-pull-request
|
Details | Review | |
59 bytes,
text/x-github-pull-request
|
Details | Review | |
1.66 MB,
image/png
|
Details | |
59 bytes,
text/x-github-pull-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
From github: https://github.com/mozilla-mobile/fenix/issues/7861.
Original bug filed under GeckoView by @itiel. Screenshots attached to original bug to demonstrate.
Currently, (almost) everywhere in the UI the alignment of varios text elements is decided based of the text directionality.
This usually makes sense, but this an issue where English text actually should appear.
See the attached screenshots with examples for this.While it's possible to address each text element separately, I don't see any reason (well, from RTL POV anyway) not to make the text align to the left for LTR, and to the right for RTL.
┆Issue is synchronized with this Jira Task
Change performed by the Move to Bugzilla add-on.
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Comment 2•2 years ago
|
||
Updated•2 years ago
|
Comment 3•1 year ago
|
||
Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/548379df0165436f3e785c4efa0574d1ff406a02
[main] Bug 1815637 - Add rtl support
Updated•1 year ago
|
Updated•1 year ago
|
Comment 4•1 year ago
|
||
Verified as fixed on Nightly 115.0a1 from 05/10 with Google Pixel 7 PRO (Android 13) and Motorola Moto G9 plus (Android 11). The alignment of text is aligned to the right for RTL.
Updated•1 year ago
|
Comment 5•1 year ago
|
||
Comment 6•1 year ago
|
||
Comment 7•1 year ago
|
||
Assignee | ||
Comment 9•1 year ago
|
||
it was a pretty risk update, I think we would like take a bit more time to be sure that we haven't introduced any problems :)
Updated•1 year ago
|
Comment 10•1 year ago
|
||
Comment 11•1 year ago
|
||
Authored by https://github.com/Amejia481
https://github.com/mozilla-mobile/firefox-android/commit/787ad6e5f46d68de09a644cbe7b2c4f8b9e049db
[main] Reverts "Bug 1815637 - Add rtl support."
Updated•1 year ago
|
Comment 12•1 year ago
|
||
Since nightly and release are affected, beta will likely be affected too.
For more information, please visit BugBot documentation.
Comment 13•1 year ago
|
||
:mavduevskiy - since this got reverted to fix bug 1839239, should it:
- be revisited to try to fix it in a way that doesn't lead to the regression?
- be left alone for the time being (e.g. applying a P3 priority and coming back to it sometime in the future)?
- Be closed as
wontfix
?
(based on what you know about this bug, your original fix, and the regression that Arturo addressed)
Comment 14•1 year ago
|
||
Comment 15•1 year ago
|
||
Comment 16•1 year ago
|
||
Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/ec16fbbde11627db5662c7b36898d32ca774120e
[releases_v116] Reverts "Bug 1815637 - Add rtl support." (#2849)
Comment 17•1 year ago
|
||
This was backed out of 117 nightly & 116 beta in bug 1839239
Assignee | ||
Comment 18•1 year ago
|
||
:jmahon
Yes, I think the patch should be iterated with the new info coming from the regression bug.
The regression bug appears to be localized. Most of dialogs are based on AlertDialog which causes no issues, but this particular case was using a TimePickerDialog. I was able to fix the crash by playing around with lifecycle – apparently I was adjusting the button position too early for the time picker.
Updated•1 year ago
|
Comment 19•1 year ago
|
||
Authored by https://github.com/mergify[bot]
https://github.com/mozilla-mobile/firefox-android/commit/ec7317b07a52bfda53a6ea74c32fffec86bb0589
[releases_v115] Reverts "Bug 1815637 - Add rtl support." (#2850)
Updated•1 year ago
|
Updated•1 year ago
|
Updated•1 year ago
|
Comment 20•1 year ago
|
||
Comment 21•1 year ago
|
||
Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/8fd38cae82759585482a1ae74be7f246ac092839
[main] Bug 1815637 - cherry picking 548379d, the original rtl commit
Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/50120cb7fe13c6459799e1740a9141985b946353
[main] Bug 1815637 – fixed the crash that was causing the 1839239 bug
Comment 22•1 year ago
|
||
The patch landed in nightly and beta is affected.
:mavduevskiy, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.Also, don't forget to request an uplift for the patches in the regression caused by this fix.
- If no, please set
status-firefox117
towontfix
.
For more information, please visit BugBot documentation.
Comment 23•1 year ago
|
||
Verified as fixed on Nightly 118.0a1 from 07/08 with Google Pixel 7 PRO (Android 14) and Motorola Moto G9 plus (Android 11). The alignment of text is aligned to the right for RTL.
Comment 24•1 year ago
|
||
Updated•1 year ago
|
Comment 25•1 year ago
|
||
Assignee | ||
Comment 26•1 year ago
|
||
Comment on attachment 9348928 [details] [review]
[mozilla-mobile/firefox-android] Bug 1815637 – RTL support (backport #3016) (#3268)
Beta/Release Uplift Approval Request
- User impact if declined: RTL languages won't have full support in the App
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): We have landed the patch before, and after making it into release it started crashing in a very particular case (older devices, upon user interaction with the webpage content). we could thoroughly – though manually – test that case and make sure it works on a the range of android versions
- String changes made/needed:
- Is Android affected?: Yes
Comment 27•1 year ago
|
||
Comment on attachment 9348928 [details] [review] [mozilla-mobile/firefox-android] Bug 1815637 – RTL support (backport #3016) (#3268) Approved for 117.0b8.
Comment 28•1 year ago
|
||
Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/a0a2fc0bbbd3831c323e10e483081a4450ece0f8
[releases_v117] Bug 1815637 - cherry picking 548379d, the original rtl commit
Authored by mike a
https://github.com/mozilla-mobile/firefox-android/commit/ef5c2fefc85676e6aab577dadf281b1a2bf5322b
[releases_v117] Bug 1815637 – fixed the crash that was causing the 1839239 bug
Comment 29•1 year ago
|
||
Verified as fixed on the latest Beta build (117.0b8).
The text is aligned to the right side, as expected.
Devices used:
- Google Pixel 7 (Android 14).
- Oppo Find X5 (Android 13).
- Lenovo M10 (Android 10).
- Samsung Galaxy S9 (Android 8).
Marking the ticket as verified for 117 as well.
Updated•1 year ago
|
Description
•