No Code Attached Yet
avatar coolcat-creations
coolcat-creations
1 Sep 2022

What needs to be fixed

The Keyboard Shortcuts are announced everywhere with J + X - but that means in Shortcut language to press the buttons simultaneos. It should be written J, X or J » Y to make it more obvious that they don't have to be pressed together.

Why this should be fixed

Already so much questions on Facebook and from Clients that say it does not work because they do whats written and not what is correct.

How would you fix it

write one of these:
J, X
J » Y
(tap 1) J (tap2) X

Side Effects expected

better understanding of what to do

avatar coolcat-creations coolcat-creations - open - 1 Sep 2022
avatar joomla-cms-bot joomla-cms-bot - change - 1 Sep 2022
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 1 Sep 2022
avatar brianteeman
brianteeman - comment - 1 Sep 2022

github and gmail both use the same type of sequential shortcut key

github just shows the two letter eg J X
gmail uses a then eg J then X

image

image

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

I like the approach with "then"
"+" is misleading and not writing anything is not clear enough i think

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

Should I create a PR with then instead of + or does something speak against it?

avatar toivo
toivo - comment - 1 Sep 2022

BTW, in Joomla 4.2.1 shortcuts only upper case letters work, as I found out by trial and error. That is probably a good thing, as long as it is clearly documented, at least in the Help section of the Keyboard Shortcuts plugin,

avatar bembelimen
bembelimen - comment - 1 Sep 2022

Should I create a PR with then instead of + or does something speak against it?

Yes, please do so

avatar brianteeman
brianteeman - comment - 1 Sep 2022

BTW, in Joomla 4.2.1 shortcuts only upper case letters work, as I found out by trial and error. That is probably a good thing, as long as it is clearly documented, at least in the Help section of the Keyboard Shortcuts plugin,

no they work in both upper and lower case

avatar toivo
toivo - comment - 1 Sep 2022

True, further test confirms they work. My first test results from PEBCAK.

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

I just prepare the PR but the language Change in the administrator/language/en-GB/plg_system_shortcut.ini and administrator/language/en-GB/plg_system_shortcut.sys.ini have no effect on the information in the modal window. Searching through the whole Project I only found a es6 File where the Keyboards are mentioned. Are they hardcoded there? @fancyFranci ?

avatar brianteeman
brianteeman - comment - 1 Sep 2022

@richard67 thats the correct place. So you can either go the github option and remove the + in the js
or create a language string
in the php pass it to the script
and then use it in the script

avatar richard67
richard67 - comment - 1 Sep 2022

@richard67 thats the correct place. So you can either go the github option and remove the + in the js or create a language string in the php pass it to the script and then use it in the script

@brianteeman Hmm, as far as I can see, the '+' at the place which I've found is used to separate the particular keys. But where does the + in the modal display come from?

avatar brianteeman
brianteeman - comment - 1 Sep 2022

Sorry I am getting really confused here (for a change)

The Keyboard Shortcuts are announced everywhere with J + X

where? Thats not what I am seeing

image

image

avatar richard67
richard67 - comment - 1 Sep 2022

I trusted the desription in the issue. I haven't checked the modal myself because I'm in the office at my regular job. But I'm confused now, too.

avatar Fedik
Fedik - comment - 1 Sep 2022

On dashboard, scroll down
image

avatar richard67
richard67 - comment - 1 Sep 2022

Ah there it is.

avatar Fedik
Fedik - comment - 1 Sep 2022

I like the idea about replacing + to then, also would do it in modal popup

avatar brianteeman
brianteeman - comment - 1 Sep 2022

@Fedik that string is a 30 second change

PLG_SYSTEM_SHORTCUT_OVERVIEW_HINT="<span class=\"icon-keyboard fa-keyboard me-2\" aria-hidden=\"true\"></span> <kbd>J</kbd> + <kbd>X</kbd> <small>Keyboard Shortcuts</small>"

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

I replaced it there but I see no change. I can upload the PR anyway.

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

@coolcat-creations On a quick search I found a hard-coded usage of '+' as separator for keys: https://github.com/joomla/joomla-cms/blob/4.2-dev/build/media_source/plg_system_shortcut/js/shortcut.es6.js#L106

I know, that was not what you are looking for. But I thought I should mention it because you might have to change that, too.

thats what I meant in my previous post

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

@Fedik that string is a 30 second change

PLG_SYSTEM_SHORTCUT_OVERVIEW_HINT="<span class=\"icon-keyboard fa-keyboard me-2\" aria-hidden=\"true\"></span> <kbd>J</kbd> + <kbd>X</kbd> <small>Keyboard Shortcuts</small>"

But the PLG_SYSTEM_SHORTCUT_XML_DESCRIPTION does not have any effect on the modal

avatar brianteeman
brianteeman - comment - 1 Sep 2022

the modal doesnt have the + you were complaining about in your first post

avatar coolcat-creations
coolcat-creations - comment - 1 Sep 2022

Yes, it does not have the "then" as well, I wanted to change it there too.

avatar Fedik Fedik - close - 1 Sep 2022
avatar Fedik
Fedik - comment - 1 Sep 2022

Closing, because we have a PR #38667
Please test

avatar Fedik Fedik - change - 1 Sep 2022
Status New Closed
Closed_Date 0000-00-00 00:00:00 2022-09-01 14:49:53
Closed_By Fedik

Add a Comment

Login with GitHub to post a comment