#29558 closed defect (bug) (fixed)
Alt+Shift shortcuts override character input
Reported by: | Balkana | Owned by: | azaozz |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | major | Version: | 2.5 |
Component: | TinyMCE | Keywords: | has-patch |
Focuses: | Cc: |
Description
I am using default wordpress editor for posts and I run into a issue when I want to write this char: " [ ".
I am from Croatia, and key combination for this is Alt Gr + F When I press that the editor goes full screen, what is really weird and I can't find a solution (I am forced to copy/paste that char from somewhere else).
Attachments (9)
Change History (38)
#2
@
10 years ago
- Keywords needs-patch added
- Severity changed from normal to major
- Version changed from 3.9.2 to 2.5
This is actually a really annoying problem. I've never noticed this before, but I also have problems when I try to add characters in TinyMCE with Alt+Shift
. A lot of those combinations are overwritten by WordPress, such as Alt+Shift+D = ∆
and Alt+Shift+L = |
(AZERTY) which I use quite a lot outside WordPress.
All these characters are blocked for an AZERTY keyboard:
ı Í ∆ ª Œ Ω › ∏ ⁄ | ¢ ‚ Æ ∑ Ó ™ Î Å
And these for QWERTY:
ˆ Ô Î Ë Ø Å Û ∏ Ù Ò Ç Â Œ Í Ú Ê Ó „
Of course that doesn't really matter. It's different for other keyboards and the user should be able to add these characters just like everywhere else.
These shortcuts are added by WordPress, not TinyMCE.
Since [7147].
#3
@
10 years ago
- Milestone changed from Awaiting Review to 4.2
- Summary changed from Key conflict in editor while using Croatian keyboard to Alt+Shift shortcuts override character input
Let's do something about this. Alt+Shift
should be replaced with something else as it is used for character input.
#4
@
10 years ago
Don't think we can change the modifier keys. Every modifier combination is already in use. The Alt+Shift+letter
was chosen as it is "reserved" in most browsers to trigger the accesskey
attribute for buttons.
Seems the only thing we can do is have an "Enable/Disable custom shortcuts" global option replacing the current one which is only for moderating comments (think there was an old ticket for that). Or maybe it can be per screen.
#5
@
10 years ago
On a mac + Chrome, you need to use ctrl+alt
for the accesskey
attribute. shift+alt
is reserved for characters.
This is the best I could find for now: https://en.wikipedia.org/wiki/Access_key
This will be fun to figure out. :)
#6
@
10 years ago
Looks like all major browsers on a mac now use ctrl+alt
for accesskey
. On window it's always either alt
or shift+alt
. So I assume alt
and shift+alt
are not used for characters on Windows? And if they do, which has precedence when the focus is an input field? Maybe we can switch to ctrl+alt
for a mac?
#7
@
10 years ago
Blocked because of this: https://github.com/tinymce/tinymce/blob/master/js/tinymce/classes/Shortcuts.js#L32
Asked spocke for suggestions. Alternatively we could use cmd+alt
, but would be better to use the same as accesskey
.
#8
@
10 years ago
We could probably get rid of the original one for link, since it's also now cmd-K. Will be a little annoying for people to relearn, I suppose.
#11
@
10 years ago
On windows there are 2 Alt keys and they work in different ways. Left Alt is used for shortcuts and right one (also with Shift modifier) is used for inserting special characters.
#12
@
10 years ago
This was added in TinyMCE: https://github.com/tinymce/tinymce/commit/7de8b6ce3578b063154148c840642d9fd8e7c503. If we are going to change the custom shortcuts on Mac, lets do it now/early (we can update to the latest dev. version of MCE).
@drozdz, unfortunately the browsers cannot detect which Ctrl/Alt/Shift key is pressed, left or right.
#13
follow-up:
↓ 14
@
10 years ago
Oh, that's great! :) But we'll need to wait for a release, right?
#14
in reply to:
↑ 13
@
10 years ago
Replying to iseulde:
Well, I suppose we can update to the latest build of TinyMCE for now. There will probably be a new release before 4.2, will update again when it comes out.
#16
@
10 years ago
About Alt or AltGr on Windows, TinyMCE says the latter produces also ctrlKey, not sure this can help though:
metaKeyPressed: function(e) { // Check if ctrl or meta key is pressed. Edge case for AltGr on Windows where it produces ctrlKey+altKey states return (Env.mac ? e.metaKey : e.ctrlKey && !e.altKey); }
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
#19
@
10 years ago
Thinking this will never be fully fixed until we let the users disable these shortcuts. There are just too many key combinations used in far too many contexts. We should let the users decide what they prefer.
I know this is not "Decisions, not options!" but that option already exists on the Profile screen:
Keyboard Shortcuts Enable keyboard shortcuts for comment moderation. More information
We should be using that globally. Lets do that in 4.3, finally :)
#20
@
10 years ago
- Keywords has-patch added; needs-patch removed
Patch dynamically generates the keyboard shortcut modal.
Also redefines the heading shortcuts. (#30422)
To do: scroll bar needs to be positioned better.
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
#23
@
10 years ago
29558.6.patch fixes the heading shortcuts for me in FF, Chrome, and Safari. I think the new modal contents are an improvement in terms of helpful information. We're still limited by the fact that TinyMCE modals aren't responsive.
This ticket was mentioned in Slack in #core-editor by azaozz. View the logs.
10 years ago
#26
@
10 years ago
29558.7.patch fixes the shortcodes on Mac.
However redoing of the modal needs more. Not so happy with removal of all the text from it. Also needs CSS fixes that are not trivial as the styles interfere with the TinyMCE UI building. Perhaps can consider moving away from tables too.
As far as I see there is no easy solution to this. Only way would be to remove the Alt + Shift + F shortcut. I'm open to any other suggestions :)