Page MenuHomeWildfire Games

Set alternative hotkey for easier tab switch
ClosedPublic

Authored by ffffffff on Dec 25 2017, 3:21 PM.

Details

Summary

Tab, Shift + Tab easier to switch tabs and when text input is selected still hotkeys alt+s, alt+w works to switch, but harder to grap : ).

Test Plan

test in options, summary, credits tabs

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

ffffffff created this revision.Dec 25 2017, 3:21 PM
bb added a comment.Dec 25 2017, 3:27 PM

Alternative hotkeys should be available for all settings, not only tabs. Also an arbitrary number of hotkeys should be allowed.

same feeling : )

ffffffff updated this revision to Diff 4942.Dec 25 2017, 3:33 PM

shortenlittle

ffffffff updated this revision to Diff 4943.Dec 25 2017, 3:36 PM

correctlittle

elexis added inline comments.Dec 25 2017, 4:40 PM
binaries/data/config/default.cfg
197 ↗(On Diff #4943)

^

bb resigned from this revision.Dec 25 2017, 5:45 PM

As said (partially on IRC), one can do this by adding hotkey.tab.next = "Alt+S", "Tab" (similar for prev) to the user.cfg. Thus I see no real need for this patch.

ffffffff updated this revision to Diff 4952.Dec 25 2017, 9:28 PM
ffffffff retitled this revision from Generalize Tab Hotkey Code + Alternative Hotkey to Zentralize panel switch code and set alternative hotkey.
ffffffff edited the summary of this revision. (Show Details)
This comment was removed by ffffffff.
ffffffff edited the test plan for this revision. (Show Details)Dec 25 2017, 9:29 PM
elexis retitled this revision from Zentralize panel switch code and set alternative hotkey to Centralize panel switch code and set alternative hotkey.Dec 26 2017, 12:41 AM
ffffffff edited the summary of this revision. (Show Details)Dec 26 2017, 4:56 PM
ffffffff retitled this revision from Centralize panel switch code and set alternative hotkey to [Centralize panel switch code and set alternative hotkey].Dec 26 2017, 7:38 PM
ffffffff edited the summary of this revision. (Show Details)
ffffffff removed a reviewer: bb.
ffffffff removed a subscriber: elexis.
ffffffff updated this revision to Diff 5037.Jan 2 2018, 4:57 PM
ffffffff retitled this revision from [Centralize panel switch code and set alternative hotkey] to Set alternative hotkey for easier tab switch.
ffffffff edited the summary of this revision. (Show Details)
ffffffff edited the test plan for this revision. (Show Details)
ffffffff added a reviewer: bb.
ffffffff edited the summary of this revision. (Show Details)
bb resigned from this revision.Jan 2 2018, 5:45 PM

Not convinced this should make it into the vanilla game, however it is a nice PoC for everyone's user.cfg

elexis added a comment.Jan 2 2018, 5:48 PM

I either missed or forgot the IRC conversation, switching tabs with tab and shift+tab doesn't seem too bad, does it? It only doesn't work if a text input is focused, no?

(I guess what we actually want is tab and shift+tab to change focus between arbitrary GUI elements, but that will likely never happen)

ffffffff added a comment.EditedJan 2 2018, 5:59 PM

It's rather more convenient to switch than the alternative hotkey alt+s or alt+w as they only are the hotkeys because they working in text input fields.

ffffffff edited reviewers, added: elexis; removed: bb.Jan 6 2018, 10:02 PM

I fear having Alt+S, Alt+W hitted on focused text input field due to it leads to changed text in that field, while switching! tab, so u dont even see ur changed text in that field f. e. in options gui!

With tab as hotkey its clear its grabbing tab hotkey for input text field either make autocomplete or do nothing. So it's not changing anything in the text input and then switch! tab. this is severe.

This is for any hotkey within text field highly severe. Text input filed should grabbing all hotkeys when focused. So it's clear while focused it cant be changed anything else around.

After that tab key and shift+tab for next and previous is very easy to hit then for tabs switching and maybe should be changed to.

ffffffff updated the Trac tickets for this revision.Jan 18 2018, 7:13 AM
elexis accepted this revision.Jan 22 2018, 5:49 AM

With the new gamesetup UI in D1027 we definitely need this hotkey in my opinion. The current default is not very intuitive and doesn't work with some window managers like mine. Sure users can change the config, but at least in alpha 23, they have to go through the edit process which is troublesome.
The hotkey eases the UI interaction tremendously, at least on my end.
Thanks for the patch fpre!

(There is no conflict with the chat in the gamesetup as the hotkeys are only triggered if one isn't typing in an input field.)

This revision is now accepted and ready to land.Jan 22 2018, 5:49 AM

Will just change the order to have the more common key combination first.

As mentioned in the ticket, this doesn't fix the bug described in the ticket and I don't commit things to hide bugs.
I just happen to prefer this hotkey.

This revision was automatically updated to reflect the committed changes.
Owners added a subscriber: Restricted Owners Package.Jan 22 2018, 6:07 AM