#21414 closed enhancement (wontfix)
Use the "Keyboard Shortcuts" checkbox in the user profile to turn on/off all custom shortcuts
Reported by: | azaozz | Owned by: | azaozz |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | |
Component: | TinyMCE | Keywords: | ux-feedback has-patch |
Focuses: | accessibility | Cc: |
Description
Custom keyboard shortcuts can be very handy not only for better accessibility but for faster access to many actions that usually require "mousing". For example we could potentially add shortcuts to access the top menu items or the more commonly used toolbar items.
However many of the available keystrokes (modifier [+ modifier] + letter) are already in use. Even worse, different shortcuts are in use in different browsers and different OS.
It would be good to let the users decide whether to enable them. As we already have a checkbox for the comment moderation shortcuts, best would be to make it "global". For now that would only affect TinyMCE.
Attachments (4)
Change History (27)
#4
@
11 years ago
- Component changed from Accessibility to TinyMCE
- Focuses accessibility added
For the record, not sure I love this.
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
9 years ago
#6
@
9 years ago
Keyboard shortcuts are terribly useful and I'm always surprised how many people are not more aware of them.
However, as a screen reader user, they can be somewhat of a nuisance since they can so easily conflict with the commands already in place. And since these are different depending on what screen reader you might be using, plus the browser and OS, I think it's very important to have a toggle to disable them.
As it is in all the time I've used WP, I have never had the comment shortcuts enabled. Very wacky things happen.
This ticket was mentioned in Slack in #accessibility by rianrietveld. View the logs.
9 years ago
#8
@
9 years ago
- Keywords close added
However many of the available keystrokes (modifier [+ modifier] + letter) are already in use. Even worse, different shortcuts are in use in different browsers and different OS.
Is this still relevant? I think we resolved all conflicts?
#9
@
9 years ago
Not sure if overall this is all that relevant anymore, but I do think that this option should be in profile/personal settings instead of a "screen option" - we use screen options for a mix of things, but generally they are about the display of the screen, and it's pretty strange to set personal preferences in multiple places. I would not want to see one global keyboard shortcuts option in default UI, though.
This ticket was mentioned in Slack in #accessibility by rianrietveld. View the logs.
9 years ago
#11
@
9 years ago
- Keywords needs-refresh added; close removed
- Milestone changed from Awaiting Review to 4.6
- Owner set to azaozz
- Status changed from new to assigned
Discussed a bit in today's accessibility bug scrub meeting, this is something we'd recommend to introduce in 4.6. Especially screen reader users may experience conflicts with existing shortcuts and it just make sense to add an option in profile/personal settings near the existing ones for comments.
Milestoning for 4.6 and assigning to @azaozz :) Any thoughts more than welcome.
This ticket was mentioned in Slack in #core by chriscct7. View the logs.
9 years ago
#13
@
9 years ago
The discussion so far:
- We resolved (nearly?) all of the conflicts with the Visual editor shortcuts.
- It will be really good to let the users disable certain shortcuts, especially for accessibility / screen reader users.
- The setting should not be in Screen Options as "they are about the display of the screen".
- We shouldn't have only one setting for all shortcuts.
- It "makes sense to add an option in profile/personal settings near the existing ones for comments".
Conclusion: we should add another checkbox (probably) after the "Enable keyboard shortcuts for comment moderation." on the Profile page, on by default, to control the extra editor shortcuts we add to TinyMCE.
One "side" question is if we should disable only the shortcuts we are adding, or all custom shortcuts like Ctrl+Shift+F for fullscreen, and leave only the default OS shortcuts like Cmd/Ctrl+C (copy), Cmd/Ctrl+V (paste), etc.
#14
@
9 years ago
In 21414.1.patch: refreshed 21414.patch, added missing bits and removed obsolete code.
Instead of removing these shortcuts from the editor help, added a note about them being disabled and a link to the profile screen to enable them (the text may need improving).
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #core by ocean90. View the logs.
8 years ago
#18
@
8 years ago
- Keywords has-patch added
- Milestone changed from 4.6 to Future Release
21414.1.patch needs a refresh to apply cleanly. Also:
- The fieldset block needs some whitespaces.
https://codex.wordpress.org/Keyboard_Shortcuts
should be its own stringCurrently disabled. Can be enabled in your %1$sprofile settings%2$s.
should beCurrently disabled. Can be enabled in your <a href="%s">profile settings</a>.
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #core-editor by afercia. View the logs.
8 years ago
This ticket was mentioned in Slack in #accessibility by afercia. View the logs.
7 years ago
#22
follow-up:
↓ 23
@
7 years ago
- Keywords needs-refresh removed
- Milestone Future Release deleted
- Resolution set to wontfix
- Status changed from assigned to closed
Closing as per today's bug scrub. Discussion can continue on closed ticket. With the advent of Gutenberg many things will change, it doesn't male much sense to introduce this feature just for a few months.
21414.patch: