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

Add keybind to toggle interaction menu #5095

Merged
merged 1 commit into from
May 5, 2017
Merged

Conversation

PabstMirror
Copy link
Contributor

@PabstMirror PabstMirror commented Apr 21, 2017

@HeadTraumaZA
Ref #3594 (comment)

Adds seperate keybind that just toggles the interaction menu.
When combined with the "Do action when releasing key: No" setting, it should be much easier to use.

Should we add new keybinds, or just add a setting to change behaviour of the existing keybinds?

@PabstMirror PabstMirror added kind/enhancement Release Notes: **IMPROVED:** kind/feature Release Notes: **ADDED:** labels Apr 21, 2017
@PabstMirror PabstMirror added this to the 3.10.0 milestone Apr 21, 2017
@CorruptedHeart
Copy link
Member

Considering Arma's method of having toggle as a separate keybind that would be consistent with Arma.

Copy link
Member

@CorruptedHeart CorruptedHeart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I gave this a test, working as expected. Even with "Do action on releasing key" it works nicely.

@rebelvg
Copy link
Contributor

rebelvg commented May 2, 2017

Tried this, pressing esc hides the menu (if show cursor is enabled) but leaves the black-screen (if enabled, of course). Without cursor enabled esc just opens esc-menu. Would be nice if esc just closed the action menu in both cases.

@PabstMirror PabstMirror merged commit c001be5 into master May 5, 2017
@PabstMirror PabstMirror deleted the interactionKeyToggle branch May 5, 2017 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement Release Notes: **IMPROVED:** kind/feature Release Notes: **ADDED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants