-
-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
Alt+L in editor triggers numbered list #8687
Comments
Begging for fix in rc2 :D |
@PaszaVonPomiot I'm not able to replicate this - for me Cmd-L toggles unordered list and Cmd-Alt-L toggles ordered list. Alt-L outputs Are you on Windows per chance? |
@kevinansfield yes, I'm a Windows 10 user. |
Hmm, I've just checked on windows 10 Chrome/Edge and can't reproduce, it's possible that's related to how my keyboard is mapped via my VM though. Ctrl-L and Ctrl-Alt-L worked fine for the list toggles 🤔 |
@kevinansfield the problem is that RAlt-L should not toggle list for selected paragraph. On English RAlt-L does not enable the list but on Polish it does. |
@PaszaVonPomiot I fully understand that for you this is an incredibly annoying bug, however it appears to be very specific to your language/environment. We're now only fixing urgent bugs before the release of 1.0.0, if you'd like to submit a fix, we can review and merge it for the first post 1.0.0 bugfix release :) Otherwise we'll get to it as soon as we can. |
The same issue is with Romainan keyboard ALT-L and ALT-P (this one publishes the post instead of inserting "§") character. |
The editor content shortcuts are handled in our fork of simplemde https://github.com/kevinansfield/simplemde-markdown-editor/blob/ghost/src/js/simplemde.js. Definitely open to PRs with fixes or research/links to show how it's possible to deal with different keyboard layouts in JS key event handlers. |
I won't be able to help with js as I'm not a programmer but hopefuly this will be useful. It appears that SimpleMDE has the same issue. First we have to understand that in JavaScript we use two parameters to determine which modifier key has been pressed and those are:
If we press AltGr using standard English (US) keyboard then all browsers will detect: However if we press AltGr using Polish (programmers) or Romanian keyboards then js will detect:
So the solution I think will be to check not only for key that is being pressed but also for its location. In our case the numbered list should be triggered by: Ctrl + Alt (only left) +L Useful linksSnippet to test keypresses Thanks! |
Copied proposed solution to simplemde repo - this isn't something to track here |
@JohnONolan it is unfortunate that you are closing this even for tracking as it is blocker for Poland and Romania. I mean we can't just stop using "ł" character and I see no resonable workaround so far. |
@PaszaVonPomiot it's tracked in your issue in SimpleMDE which is where this needs to be fixed, keeping multiple issues open means that communication gets split and comments such as yours above don't end up in the right place. Core team monitors our 3rd party deps as well as the core repos |
It appears that upstream markdown editor development has halted. Any ideas where to go from here? |
Yep. You could fix it, or find someone else to fix it for you, and open a pull request to SimpleMDE. |
Issue Summary
Numbered list should be triggered by Ctrl+Alt+L but they are also triggered by Alt+L
I need to use Alt+L to type "Ł" letter.
Technical details:
The text was updated successfully, but these errors were encountered: