Closed Bug 1917175 Opened 14 days ago Closed 9 days ago

Browser considers Saraiki (skr) as LTR rather than RTL

Categories

(Core :: Internationalization, defect, P1)

Firefox 130
defect

Tracking

()

VERIFIED FIXED
132 Branch
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)

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?

Flags: needinfo?(bvisness)

[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.

Set release status flags based on info from the regressing bug 1893643

Potentially, we could uplift a fix in the planned dot release (if it lands on nightly and beta first)

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.

Flags: needinfo?(bvisness)
Assignee: nobody → earo
Status: NEW → ASSIGNED
Pushed by [email protected]: https://hg.mozilla.org/integration/autoland/rev/4f29692950ab Re-update unic-langid and unic-langid-impl to 0.9.5. r=bvisness
Attachment #9423817 - Flags: approval-mozilla-beta?

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
Severity: -- → S1
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 9 days ago
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch

Please nominate this for Release approval also.

Flags: needinfo?(earo)
Attachment #9423817 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Please note that tomorrow is the deadline to nominate this fix for the planned 130 dot release.

Attachment #9424338 - Flags: approval-mozilla-release?

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
Attachment #9424338 - Flags: approval-mozilla-release? → approval-mozilla-release+

This fix will be included in our planned 130 dot release, thanks.

Flags: needinfo?(earo)
Flags: qe-verify+

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.

Status: RESOLVED → VERIFIED
Has STR: --- → yes
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: