Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

F10 is mentioned as fullscreen toggle; should be F11. #40255

Closed
FDIN-NL opened this issue Mar 30, 2023 · 5 comments
Closed

F10 is mentioned as fullscreen toggle; should be F11. #40255

FDIN-NL opened this issue Mar 30, 2023 · 5 comments

Comments

@FDIN-NL
Copy link

FDIN-NL commented Mar 30, 2023

Steps to reproduce the issue

Making an override in i.e. the template presents the codemirror as editor. Here the F10 is mentioned to switch to full screen.

Expected result

Advise the F11 as this works in a.o. Chrome and Firefox

Actual result

F10 is advised and does not work.

System information (as much as possible)

Joomla 4 current version

Additional comments

I changed my local version:

Solution:
plugins/editors/codemirror/layouts/editors/codemirror/element.php
line 32 $fskeys[] = $params->get('fullScreen', 'F10');
changed to
line 32 $fskeys[] = $params->get('fullScreen', 'F11');

@brianteeman
Copy link
Contributor

brianteeman commented Mar 30, 2023

Sorry you are mistaken about what this does.

The f11 you are referring to is a browser hotkey which makes the browser go full screen

Here the reference to f10 is a codemirror hotkey (codemirror needs to have keyboard focus) which makes codemirror take up the entire screen.

f10

@Quy
Copy link
Contributor

Quy commented Mar 30, 2023

Closing as not an issue for reason stated.

@Quy Quy closed this as completed Mar 30, 2023
@FDIN-NL
Copy link
Author

FDIN-NL commented Mar 31, 2023

Thank you Brian for explaining. I tried to find any document about CODEMIRROR and FULL SCREEN annd this came up:
https://codemirror.net/5/demo/fullscreen.html
Apparently they agree with me that F11 is the way to go.
I have retried the toggle F11 and as you explained with the focus on the codemirror sub-window it will work, and with the focus on the rest of the browser-page it also works!
With my suggested change they both work as expected.
Please reconsider.

@Quy I would appreciate to be able to close MY issue after I am satisfied with the response.

@brianteeman
Copy link
Contributor

You could of course use the EXISTING settings in the Joomla settings for codemirror that let you set the function key to whatever you want - including adding a modifier

image

@FDIN-NL
Copy link
Author

FDIN-NL commented Mar 31, 2023

Thank you again. There is still so much to discover in Joomla!

This is the perfect solution. I would not mind if this became the default...

😃


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants