RTC Language Change a11y PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
9 Sep 2024

Pull Request for Issue #44040 .

Summary of Changes

Use different language Keys for Pagination Buttons. This enables better translation.

Testing Instructions

See #44040. For english sites, The Aria Labels in pagination are correct before and after patch.
Text can then be translated correcty.
For Example German "Gehe zur nächsten Seite" statt

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

"Zur Seite weiter wechseln" is not a correct german sentence.

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar chmst chmst - open - 9 Sep 2024
avatar chmst chmst - change - 9 Sep 2024
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 9 Sep 2024
Category Administration Language & Strings Layout
avatar chmst chmst - change - 9 Sep 2024
Title
Improve translation of aria label text by extra language keys
[4.4] Improve translation of aria label text by extra language keys
avatar chmst chmst - edited - 9 Sep 2024
avatar brianteeman
brianteeman - comment - 9 Sep 2024

I am surprised at the way the current code creates the language strings. I can't see why it was created that way.
I am also surprised that no translation team has raised this issue before.

As the text in the core doesn't change and at least one language will be improved by this change then its all good by me

avatar brianteeman brianteeman - test_item - 9 Sep 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 9 Sep 2024

I have tested this item ✅ successfully on ac0e2eb


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44050.

avatar fgsw fgsw - test_item - 10 Sep 2024 - Tested successfully
avatar fgsw
fgsw - comment - 10 Sep 2024

I have tested this item ✅ successfully on ac0e2eb


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44050.

avatar brianteeman
brianteeman - comment - 10 Sep 2024

oops I missed the code change that @sandewt spotted.

avatar brianteeman brianteeman - alter_testresult - 10 Sep 2024 - brianteeman: Not tested
avatar chmst
chmst - comment - 10 Sep 2024

oops - missed it too.. thank you @sandewt

avatar chmst chmst - change - 10 Sep 2024
Labels Added: Language Change a11y PR-4.4-dev
avatar brianteeman brianteeman - test_item - 10 Sep 2024 - Tested successfully
avatar brianteeman
brianteeman - comment - 10 Sep 2024

I have tested this item ✅ successfully on 9fa3463


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44050.

avatar Quy Quy - test_item - 10 Sep 2024 - Tested successfully
avatar Quy
Quy - comment - 10 Sep 2024

I have tested this item ✅ successfully on 9fa3463


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44050.

avatar Quy Quy - change - 10 Sep 2024
Status Pending Ready to Commit
avatar Quy
Quy - comment - 10 Sep 2024

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/44050.

avatar Quy Quy - change - 10 Sep 2024
Labels Added: RTC
avatar laoneo laoneo - close - 11 Sep 2024
avatar laoneo laoneo - merge - 11 Sep 2024
avatar laoneo laoneo - change - 11 Sep 2024
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2024-09-11 05:57:12
Closed_By laoneo
avatar laoneo
laoneo - comment - 11 Sep 2024

Thanks!

Add a Comment

Login with GitHub to post a comment