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

Prevent CBA keybindings in zeus interface #2216

Merged
merged 2 commits into from
Aug 31, 2015
Merged

Conversation

kymckay
Copy link
Member

@kymckay kymckay commented Aug 20, 2015

Checks if the key pressed matches the zeus menu keybind and only fires the code to handle CBA keybinds if true. Should fix the majority of weirdness with keybinds in zeus. However, not perfect as shared keybinds with different modifiers will still fire and it's really an un-ideal workaround.

Fixes #1951

Checks if the key pressed matches the zeus menu keybind and only fires the code to handle CBA keybinds if true. Should fix the majority of weirdness with keybinds in zeus. However, not perfect as shared keybinds with different modifiers will still fire and it's really an un-ideal workaround.
@kymckay kymckay added this to the 3.3.0 milestone Aug 20, 2015
@kymckay kymckay added the kind/bug-fix Release Notes: **FIXED:** label Aug 20, 2015
The hacky keybind detection method should consider modifier keys also to prevent keybinds on the same key with different modifiers from firing
@kymckay
Copy link
Member Author

kymckay commented Aug 31, 2015

Modifier keys now taken into account, tested and fixed 😄

Also, confirming that CBA_fnc_getKeybind does return the current key rather than the default one 👍

@kymckay kymckay self-assigned this Aug 31, 2015
kymckay added a commit that referenced this pull request Aug 31, 2015
Prevent CBA keybindings in zeus interface
@kymckay kymckay merged commit 5e57cad into master Aug 31, 2015
@kymckay kymckay deleted the isolateZeusMenuKeybind branch August 31, 2015 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zeus interface climb
1 participant