Browser considers Saraiki (skr) as LTR rather than RTL
Categories
(Core :: Internationalization, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr115 | --- | unaffected |
firefox-esr128 | --- | unaffected |
firefox130 | + | verified |
firefox131 | + | verified |
firefox132 | + | verified |
People
(Reporter: eemeli, Assigned: eemeli)
References
(Regression)
Details
(Keywords: regression)
Attachments
(3 files)
48 bytes,
text/x-phabricator-request
|
Details | Review | |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-beta+
|
Details | Review |
48 bytes,
text/x-phabricator-request
|
phab-bot
:
approval-mozilla-release+
|
Details | Review |
The earlier update of the unic-langid
and unic-langid-impl
crates from 0.9.1 to 0.9.5 made in bug 1869512 was rolled back in patch D213777 for bug 1893643.
The consequence of the rollback is that we use CLDR 37 rather than CLDR 44 data for language directionality, and the earlier had Saraiki incorrectly coded as LTR rather than RTL.
Ben, it looks like this may have been accidental?
Assignee | ||
Updated•14 days ago
|
Comment 1•14 days ago
|
||
[Tracking Requested - why for this release]: while we have few users for Saraiki (it was introduced in 128), this is showing the browser with wrong text direction.
Comment 2•14 days ago
|
||
Set release status flags based on info from the regressing bug 1893643
Comment 3•14 days ago
|
||
Potentially, we could uplift a fix in the planned dot release (if it lands on nightly and beta first)
Comment 4•14 days ago
|
||
Yes, this was accidental. I only meant to update our wasm-tools packages for internal testing. I think I just carried over an unrelated update while rebasing. Apologies for that.
Assignee | ||
Comment 5•11 days ago
|
||
Updated•11 days ago
|
Assignee | ||
Comment 7•10 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D221430
Updated•10 days ago
|
Comment 8•10 days ago
|
||
beta Uplift Approval Request
- User impact if declined: Users of the Saraiki localization will have the browser render as left-to-right instead of right-to-left
- Code covered by automated testing: no
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: In language settings, change the UI language to Saraiki (سرائیکی). Note that a reload is required as it should be for a LTR -> RTL switch, and after a restart layout switches to be RTL.
- Risk associated with taking this patch: low
- Explanation of risk level: The change was already included earlier, but was accidentally rolled back
- String changes made/needed: no
- Is Android affected?: no
Updated•9 days ago
|
Comment 9•9 days ago
|
||
bugherder |
Comment 10•8 days ago
|
||
Please nominate this for Release approval also.
Updated•8 days ago
|
Updated•8 days ago
|
Comment 11•8 days ago
|
||
uplift |
Comment 12•8 days ago
|
||
Please note that tomorrow is the deadline to nominate this fix for the planned 130 dot release.
Comment 13•8 days ago
|
||
Original Revision: https://phabricator.services.mozilla.com/D221430
Updated•8 days ago
|
Comment 14•8 days ago
|
||
release Uplift Approval Request
- User impact if declined: Users of the Saraiki localization will have the browser render as left-to-right instead of right-to-left
- Code covered by automated testing: no
- Fix verified in Nightly: yes
- Needs manual QE test: no
- Steps to reproduce for manual QE testing: In language settings, change the UI language to Saraiki (سرائیکی). Note that a reload is required as it should be for a LTR -> RTL switch, and after a restart layout switches to be RTL.
- Risk associated with taking this patch: low
- Explanation of risk level: The change was already included in fx128, but was accidentally rolled back by mistake in unrelated patch
- String changes made/needed: None
- Is Android affected?: no
Updated•8 days ago
|
Updated•8 days ago
|
Comment 16•8 days ago
|
||
uplift |
Assignee | ||
Updated•8 days ago
|
Updated•3 days ago
|
I have reproduced the issue with Firefox 130.0 (20240829075237) and 132.0a1 (2024-09-06) on Windows 10x64. After changing the language to Saraiki (سرائیکی) as stated in comment 14 the UI layout will not change to RTL even after a restart.
The issue is verified fixed with Firefox 130.0.1, 131.0b7, and 132.0a1 (2024-09-16) on Windows 10x64, macOS 12, and Ubuntu 24. After changing the language to Saraiki (سرائیکی) the UI will change to RTL and will stay as expected after a restart.
Description
•