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

Mozilla shortcut hotkeys with keyboard macros (e.g., alt_ctrl_o, alt_ctrl_q) not working #58

Closed
wants to merge 18 commits into from

Conversation

hiteshramola
Copy link

@hiteshramola hiteshramola commented May 17, 2019

Mozilla shortcut hotkeys with keyboard macros (e.g., alt_ctrl_o, alt_ctrl_q) not working.

Because when pressing

Alt + Ctrl + [any character mapped with macro (q,e,r,t,y,u,i,o,a,d,g,h,l,x,n,m)]

it gives

event.altKey = false
event.ctrlKey = false

which is result gives no mapping to the mentioned hotkey alt_ctlr_o.

tzury and others added 18 commits March 10, 2010 07:52
Doesn't trigger when keypress is on an item with contentEditable set to true.
…t.data.

Some benefits:

 - Enable event delegation (closes #10, closes #11)
 - Option to unbind a specific key (closes #12)
Change the way plugin works to use namespaces instead of special data
The way it was checking for no keys was wrong since `"".split(" ")` returns
an Array with a single empty string, checking for `keys.length` isn't enough.

Bug was introduced when changed the way plugin works to use namespaces.
Fix event handler in case no key is specified.
Allow for the use of multiple namespaces.
Without this check jQuery UI's autocomplete silently fails. The remote datasource is never hit for the autocomplete data.
Don't side effect jQuery UI autocomplete fields.
Work with autocomplete jqueryUI 1.9.0
Now works with accordion, menu and tabs.
Now work with jQueryUI 1.9.0
@hiteshramola
Copy link
Author

Ignore the above I have raised a new pull request jquery.hotkeys/pull/59
Please see that for the changes.

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

Successfully merging this pull request may close these issues.

9 participants