joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[5.1]Installation Language

Open brianteeman opened this issue 2 years ago • 19 comments

Pull Request for Issue #36048 .

Summary of Changes

At present the Language Selection in Joomla Installer displays only the nativeName in the list. This makes language names to display in different places not alphabetically by [English] name. This is true especially for language names that do not contain Latin characters. For example, Spanish would appear immediately after English as the native name for Spanish is Español. You can also notice other language such as Tamil (with native name தமிழ்)is listed after the English alphabet names.

Suggestion is to show the language name (ie English Name) along with native name in the list and the list sorted in the language name. This would help to see the languages alphabetically in English. In addition, it would help to find the language easily.

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

image

Documentation Changes Required

brianteeman avatar Aug 23 '22 17:08 brianteeman

It still depends on localization teams. There are several languages where the <nativeName> and <name> headers are reversed.

Someone needs to contact the localization teams (e.g. Bulgarian, etc.) and fix this. Then we get the result you describe - "alphabetical" and "easy to find".

Just in case, I'll note @tecpromotion. Don't kick me if it's none of your business :).

Kostelano avatar Aug 23 '22 18:08 Kostelano

I raised this a long long time ago so I 10000% agree

And you pr got rejected? If you create a pr I would support it.

brianteeman avatar Oct 21 '22 17:10 brianteeman

This pull request has been automatically rebased to 5.0-dev.

HLeithner avatar May 02 '23 16:05 HLeithner

@HLeithner I made this because you asked

brianteeman avatar May 13 '23 15:05 brianteeman

@brianteeman thanks for bringing this up again, I will check and find some testers hopefully soon.

HLeithner avatar May 13 '23 15:05 HLeithner

This pull request has been automatically rebased to 5.1-dev.

HLeithner avatar Sep 30 '23 22:09 HLeithner

If this PR is never going to be accepted then please close it. I only made it at the request of the release lead

brianteeman avatar Dec 19 '23 08:12 brianteeman

alpha2 packages are already ready... @LadySolveig can you merge this at some point in time?

HLeithner avatar Dec 19 '23 11:12 HLeithner

@HLeithner @brianteeman sure - it would be a shame if it got lost. I see no tests yet. I will ask for tests again and hopefully merge soon.

LadySolveig avatar Dec 19 '23 13:12 LadySolveig

The system tests fail at installing. Can be seen here when using the "show all logs" at the top or downloading the complete log. The log then shows:

1) Install Joomla
       Install Joomla:
     CypressError: Timed out retrying after 4000ms: `cy.select()` failed because it could not find a single `<option>` with value, index, or text matching: `English (United Kingdom)`

It happens in file tests/System/integration/install/Installation.cy.js.

@brianteeman Do you think you can adapt the system tests to your change, or should we try to find someone who can help? Unfortunately I have no knowledge on Cypress.

P.S.: This should not keep people from testing, any later fix for the system tests will not influence the result of human tests.

richard67 avatar Dec 19 '23 14:12 richard67

The issue is not with the test but with the upstream joomla-cypress package node_modules\joomla-cypress\src\joomla.js

brianteeman avatar Dec 19 '23 14:12 brianteeman

The issue is not with the test but with the upstream joomla-cypress package node_modules\joomla-cypress\src\joomla.js

Yes, I meanwhile know, and @LadySolveig will handle it.

richard67 avatar Dec 19 '23 14:12 richard67

I have tested this item :white_check_mark: successfully on 186eca26a67ced80bf17cb014f9f4254d2c11310


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

richard67 avatar Dec 19 '23 14:12 richard67

I have tested this item :white_check_mark: successfully on d42fc0af79403b4f6acedf23e26161dd263e8a6b


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

richard67 avatar Dec 19 '23 15:12 richard67

I have tested this item :white_check_mark: successfully on d42fc0af79403b4f6acedf23e26161dd263e8a6b


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

J5 1_Test2

dautrich avatar Dec 19 '23 18:12 dautrich

RTC


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

richard67 avatar Dec 19 '23 18:12 richard67

I've set RTC despite of the failing system tests as @LadySolveig told me that she will adjust the tests when this PR gets merged.

richard67 avatar Dec 19 '23 18:12 richard67

I just see there is already a PR for adjusting the system tests: https://github.com/joomla-projects/joomla-cypress/pull/11

Thanks @alikon .

richard67 avatar Dec 19 '23 18:12 richard67

Any reason why this RTC pull request is not being merged?

brianteeman avatar Feb 21 '24 09:02 brianteeman

Thank you @brianteeman and for the review and for testing to @richard67 @dautrich

LadySolveig avatar Mar 14 '24 17:03 LadySolveig

yay!!

brianteeman avatar Mar 14 '24 17:03 brianteeman