Opened 7 years ago
Last modified 2 years ago
#4411 new defect
Missing confirm hotkeys
Reported by: | Imarok | Owned by: | |
---|---|---|---|
Priority: | Nice to Have | Milestone: | Backlog |
Component: | UI – Miscellaneous | Keywords: | |
Cc: | Patch: | Phab:D1046 |
Description (last modified by )
Some dialog buttons miss the confirm hotkey:
prelobby - would be really useful here!
aiconfig - would be useful!
splashscreen - absolutely missing, as it allows closing it immediately if it was read before already or wasn't cared about. Closing it with escape won't save the "read" flag, while closing with enter would.
locale - would be useful to accept
gamesetup - setting the confirmation button wouldn't work initially as the chat is focused, but wouldn't the "start the game with hotkey" ticket become solved with this setting, once the chat isn'T focused anymore?
loadgame - would be useful
msgbox - We always wanted that, but it depends on the order. So we should ping that one alpha 22 ticket we have about that in the way of adding a requirement (confirm hotkey should be set dynamically depending on the choice)
credits - could map to the only button: close, even though that is accessible with escape already.
loading - no user interaction, confirm hotkey can't be applied
gamesetup_mp - already implemented
savegame - already present, (<action on="Press"> on that text input)
replaymenu - to start the replay
structree - could be an alias for close
lobby - not really useful as there is no obvious primary choice
atlas - empty page
locale_advanced - escape present, confirm not
options - we mgiht refrain from assigning it to the save button in order to avoid unintentional saves when hitting enter after adding text input elements and losing focus of that element. Aboslutely missing the "close with escape" function here!
summary - would be useful to continue
manual - wouldn't hurt to have an alias to close the thing (in case one hand is already close the enter button, one can save few time)
civinfo - same
pregame - that's the mainmenu, not possible to determine a primary choice, perhaps opening the singleplayer menu, but that sounds like a handful of lines rather than one
session - that won't work out
related
- Ticket: #6510
Attachments (1)
Change History (17)
by , 7 years ago
Attachment: | 4411_confirm_hotkey_lobby.patch added |
---|
comment:1 by , 7 years ago
Keywords: | rfc added; review removed |
---|
comment:2 by , 7 years ago
Description: | modified (diff) |
---|
comment:3 by , 7 years ago
Keywords: | rfc removed |
---|---|
Summary: | [PATCH] Confirm hotkey for the prelobby window → [PATCH] Missing confirm hotkeys |
The diff can be committed, see refs D47.
The return button would be useful in other dialogs as well: prelobby - would be really useful here! aiconfig - would be useful! splashscreen - absolutely missing, as it allows closing it immediately if it was read before already or wasn't cared about. Closing it with escape won't save the "read" flag, while closing with enter would. locale - would be useful to accept gamesetup - setting the confirmation button wouldn't work initially as the chat is focused, but wouldn't the "start the game with hotkey" ticket become solved with this setting, once the chat isn'T focused anymore? loadgame - would be useful msgbox - We always wanted that, but it depends on the order. So we should ping that one alpha 22 ticket we have about that in the way of adding a requirement (confirm hotkey should be set dynamically depending on the choice) credits - could map to the only button: close, even though that is accessible with escape already. loading - no user interaction, confirm hotkey can't be applied gamesetup_mp - already implemented savegame - already present, (<action on="Press"> on that text input) replaymenu - to start the replay structree - could be an alias for close lobby - not really useful as there is no obvious primary choice atlas - empty page locale_advanced - escape present, confirm not options - we mgiht refrain from assigning it to the save button in order to avoid unintentional saves when hitting enter after adding text input elements and losing focus of that element. Aboslutely missing the "close with escape" function here! summary - would be useful to continue manual - wouldn't hurt to have an alias to close the thing (in case one hand is already close the enter button, one can save few time) civinfo - same pregame - that's the mainmenu, not possible to determine a primary choice, perhaps opening the singleplayer menu, but that sounds like a handful of lines rather than one session - that won't work out
comment:4 by , 7 years ago
Keywords: | simple added |
---|---|
Milestone: | Alpha 22 → Backlog |
Priority: | Should Have → Nice to Have |
comment:6 by , 7 years ago
Description: | modified (diff) |
---|---|
Keywords: | patch removed |
Summary: | [PATCH] Missing confirm hotkeys → Missing confirm hotkeys |
comment:7 by , 7 years ago
Description: | modified (diff) |
---|
Some missing close-hotkeys too, refs Phab:D813
comment:9 by , 6 years ago
Patch: | → https://code.wildfiregames.com/D1046 |
---|
comment:10 by , 6 years ago
Owner: | set to |
---|---|
Status: | new → assigned |
comment:11 by , 6 years ago
Milestone: | Backlog → Alpha 23 |
---|
comment:12 by , 6 years ago
Keywords: | simple removed |
---|---|
Milestone: | Alpha 23 → Work In Progress |
Patch: | https://code.wildfiregames.com/D1046 → Phab:D1046 |
Not essential for a23 as long as we have mouseclicks.
comment:13 by , 6 years ago
The features are less interesting than the others on the milestone, there is no reviewer for them and we aspire feature freeze.
comment:14 by , 5 years ago
Component: | UI & Simulation → Misc. UI |
---|
Move tickets to Misc. UI
as "UI & Simulation" got some sub components.
comment:16 by , 2 years ago
Milestone: | Work In Progress → Backlog |
---|---|
Owner: | removed |
Status: | assigned → new |
An empty ticket description (or review) could be compared to a triangle with 2 points. A non-empty string shows consideration, so posting one:
(For every patch, there is a way to improve and since you're asking for a review:) This patch here might be considered incomplete if there are more GUI pages where the confirm hotkey is absent (ping #3897 (focus should not be an issue as that is taken into account correctly by the engine mostly)). Since we have about a dozen of GUI pages, we can check all of them.
Dynamically setting the confirm hotkey for message boxes should be done in #4345 as it depends on the user-specified button order.