Quickicon improvements

[chmst]

Pull Request for Issue #258 .

Summary of Changes

1. Fixed: Button MenuItems

2. Numbers on icons can be useful and important but sometimes are useless, asdescribe in the issues.
   We have expanded module parameters so the administrator can choose if she want to see the number or only the button.

3. If a number is displayed, both the number and component name have attribute "aria-hidden", but a sr-only text is added which gives information in a sentence.

[brianteeman]

plugins? do you mean extensions? Is this even used?

[chmst]

We don't have icons for languages and extensions. So better delete the language keys

[brianteeman]

Made some comments inline

Someone else will have to check that the pluralisation is correct in the code to allow for languages that have more complex pluralisation rules - maybe @infograf768 can check this

[chmst]

The Plural forms do this:

1. for sighted users

2. For blind users the Screen Reader says for example:

Users: One User is activated.

Articles: No article is available

Modules: Three modules are available

Global Checkin: No Items are locked.

Cache: The Cache is empty

We have adopted this from the joomla update plugins. In case of many different plural forms we could add more strings.

[brianteeman]

In case of many different plural forms we could add more strings.

That's the whole point - we can not. The current system used throughout joomla is that the TT add them if they need to

[bembelimen]

@infograf768 you're right, we have to take care for the pluralism form. Do you have an JS example where this happen in Joomla!?

[infograf768]

Did not find any but this should help https://github.com/joomla/joomla-cms/blob/4.0-dev/libraries/src/Language/Text.php#L171-L198

[infograf768]

@chmst
Please use the normalized string constants for plural forms.
I mean adding the _N_ in the constant. For example:
COM_CATEGORIES_QUICKICON_N_SRONLY

[chmst]

Thank ou, @infograf768.
I did the normalization but was not successful with the RTL. Will try again when I have some spare time.

[infograf768]

RTL: will test a new install of your branch and see what I can do.

[infograf768]

RTL: the problem with the Cache value display is the JsonResponse.
It does not accept htmlentities in general and even if it was, this specific one should not be decoded anyway as it is a pure unicode ‎ = \u200E

For example if I use
$result['amount'] = json_encode('‎' . $mb);
I get the correct LTR but with the htmlentity

[infograf768]

@chmst
RTL: found a solution in quickicon.es6.js
Change
counter.textContent = response.data.amount;
to
counter.textContent = '\u200E' + response.data.amount;

Same for sronly
sronly.textContent = '\u200E' + response.data.sronly;

provided that RTL TT creates a string formatted such:
COM_CACHE_QUICKICON_SRONLY= "%s حجم کش است"