? ? Pending

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
7 May 2020

Summary of Changes

When one is using various languages with different country codes (and also when a specific language is not using the language icon), differentiating them in admin may be hard.

For example:
de-DE, de-AT, de-CH
en-GB, en-US, en-AU
fr-FR, fr-CA

etc.

The badge produced are using the language sef (URL language code) as defined in the content language but it is confusing if that sef is not simple.
I propose here to use the full language code instead as it is clearer.
It also normalises the language all over admin (See below the Administrator menu MENUS for menus containing homes)

Screen Shot 2020-05-07 at 12 17 02

These show in various managers (associations column, home menu item, com_associations, ...)

I am conscious that this will use more space when many languages but I think it is worth it.

Testing Instructions

Create a multilingual site and display the various managers

Before patch

Screen Shot 2020-05-07 at 11 35 45

After patch

Here the German de-DE content language has no image/flag

Screen Shot 2020-05-07 at 11 36 49

avatar infograf768 infograf768 - open - 7 May 2020
avatar infograf768 infograf768 - change - 7 May 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 7 May 2020
Category Administration com_associations com_categories com_contact com_content com_menus com_newsfeeds com_templates
avatar infograf768 infograf768 - change - 7 May 2020
The description was changed
avatar infograf768 infograf768 - edited - 7 May 2020
avatar richard67
richard67 - comment - 7 May 2020

The "After Patch" screenshot would look better if all the badges for the languages would have the same width.

avatar richard67
richard67 - comment - 7 May 2020

Mail templates still show the 2 letter code if for a content language the flag is set to "None".

avatar richard67
richard67 - comment - 7 May 2020

If you would add a css class to the html only for the language badges, you could give them all same width with some (s)css. Since the language codes aren't translated, we could just make sure that a "ww-WW" fits into it (W because widest character in most common fonts).

avatar infograf768
infograf768 - comment - 7 May 2020

@richard67
we also have lang tags of type xxx-XX
therefore better let the width auto imho.

will look at mail templates tomorrow.

avatar richard67
richard67 - comment - 7 May 2020

@infograf768 Then I would make them all big enough for "www-WW". The auto width really looks bad. But that's just my opinion, I would not block this PR because of it.

If you want I can do the mail template changes for you. I've tested them here. Shall I? And if so, shall I make PR to your branch, or shall I commit directly to your branch? If you don't want me to do it: No problem, we can wait until tomorrow, no hurry. Just wanted to offer help if you are off desktop right now.

avatar brianteeman brianteeman - test_item - 7 May 2020 - Tested successfully
avatar brianteeman
brianteeman - comment - 7 May 2020

I have tested this item successfully on 83f9fa0


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

avatar infograf768 infograf768 - change - 8 May 2020
Labels Added: ?
avatar infograf768
infograf768 - comment - 8 May 2020

@richard67
com_mails done
Screen Shot 2020-05-08 at 07 41 53

avatar infograf768
infograf768 - comment - 8 May 2020

Concerning the badge width I stand with the auto. This PR is already increasing the size of these badges. Giving them an important fixed width would be counterproductive and not add to legibility imho.

avatar richard67
richard67 - comment - 8 May 2020

Maybe the badges can be aligned differently so they look better together with different width? Anyway, not to be done by this PR, should be done separately, if something can be done.

avatar richard67 richard67 - test_item - 8 May 2020 - Tested successfully
avatar richard67
richard67 - comment - 8 May 2020

I have tested this item successfully on c032365


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

avatar infograf768
infograf768 - comment - 8 May 2020

@brianteeman
Please test again.

avatar brianteeman brianteeman - test_item - 8 May 2020 - Tested successfully
avatar brianteeman
brianteeman - comment - 8 May 2020

I have tested this item successfully on c032365


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

avatar infograf768 infograf768 - change - 8 May 2020
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 8 May 2020

RTC. Thanks for testing.


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

avatar wilsonge wilsonge - change - 8 May 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-05-08 22:34:50
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 8 May 2020
avatar wilsonge wilsonge - merge - 8 May 2020
avatar wilsonge
wilsonge - comment - 8 May 2020

Thanks!

Add a Comment

Login with GitHub to post a comment