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

Keyboard shortcuts vs accesskeys #38

Open
ba55ie opened this issue Jul 29, 2016 · 4 comments
Open

Keyboard shortcuts vs accesskeys #38

ba55ie opened this issue Jul 29, 2016 · 4 comments
Assignees
Labels

Comments

@ba55ie
Copy link

ba55ie commented Jul 29, 2016

The keyboard shortcuts aren't working on my local installation (patternlab-node v2.3.0) and online. Tested with the latest Chrome and Firefox.

My suggestion is to use accesskeys instead of jwerty, because it will make the site more accessible and you can ditch the JS dependency.

https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/accesskey

@ba55ie
Copy link
Author

ba55ie commented Jul 29, 2016

Ah, according to the docs this feature is disabled, but it seems to be active in the code:

jwerty.key('ctrl+shift+s', function(e) {

@ba55ie ba55ie changed the title Keyboard shortcuts aren't working Keyboard shortcuts - accesskeys Jul 29, 2016
@ba55ie ba55ie changed the title Keyboard shortcuts - accesskeys Keyboard shortcuts vs accesskeys Jul 29, 2016
@dmolsen
Copy link
Member

dmolsen commented Jul 29, 2016

@ba55ie -

Some of them are in there but not all. I'll look into accesskey as well. Thanks for the heads up.

@dmolsen dmolsen self-assigned this Jul 29, 2016
@dmolsen dmolsen added the bug label Jul 29, 2016
@bradfrost
Copy link
Member

I asked around about this: https://twitter.com/brad_frost/status/917429730630098951 and got some interesting results from a11y folks. It sounds like accesskeys can interfere with screen readers' built-in keyboard shortcuts. I'm going to ponder this a bit more, but in any case, I think moving to a non-jQuery version (referenced in #78) is a good idea.

@ba55ie
Copy link
Author

ba55ie commented Oct 10, 2017

OK, thanks for the heads up. Too bad that the usage of access keys still causes problems after so many years...

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

No branches or pull requests

3 participants