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

More dialogs to exit with READY hotkey #4443

Merged
merged 2 commits into from
Oct 19, 2021

Conversation

a1exsh
Copy link
Contributor

@a1exsh a1exsh commented Oct 17, 2021

Hero, Castle and Kingdom dialogs have only one button: EXIT. Especially
when playing with one hand holding the mouse (to the right of the keyboard) it
is more convenient to exit these screens by hitting the num-pad ENTER.

Otherwise one has to either use the mouse to click on the button or use one's
left hand to hit ESC.

Hero, Castle and Kingdom dialogs have only one button: `EXIT`.  Especially
when playing with one hand holding the mouse (to the right of the keyboard) it
is more convenient to exit these screens by hitting the num-pad `ENTER`.

Otherwise one has to either use the mouse to click on the button or use one's
left hand to hit `ESC`.
@a1exsh
Copy link
Contributor Author

a1exsh commented Oct 17, 2021

Also found and updated these dialogs:

  • Battle options
  • Adventure options
  • File options
  • Army info
  • View world
  • Spell book

@oleg-derevenetz oleg-derevenetz added improvement New feature, request or improvement ui UI/GUI related stuff labels Oct 18, 2021
@oleg-derevenetz oleg-derevenetz added this to the 0.9.9 milestone Oct 18, 2021
@ihhub
Copy link
Owner

ihhub commented Oct 19, 2021

Hi @a1exsh , let's think from consistency point of view: if a dialog contains OKAY button then using Enter makes sense as this key is used as a confirmation. When a dialog contains EXIT button Esc (Escape --> exit) is more appropriate. So I might suggest to stick to this logic rather than trying to adopt to everyone's style. What do you think?

I'd like to hear @Branikolog and @ LeHerosInconnu opinions as well.

@a1exsh
Copy link
Contributor Author

a1exsh commented Oct 19, 2021

For me personally, convenience is one of the major reasons to play fheroes2 — otherwise I'd just stick with the OG ;)

@ihhub
Copy link
Owner

ihhub commented Oct 19, 2021

For me personally, convenience is one of the major reasons to play fheroes2 — otherwise I'd just stick with the OG ;)

I absolutely agree but consistency is important as well. Imagine if an old player from HoMM2 comes and tries to use Esc button and realizes that it doesn't work :)

@a1exsh
Copy link
Contributor Author

a1exsh commented Oct 19, 2021

For me personally, convenience is one of the major reasons to play fheroes2 — otherwise I'd just stick with the OG ;)

I absolutely agree but consistency is important as well. Imagine if an old player from HoMM2 comes and tries to use Esc button and realizes that it doesn't work :)

Ah, about that you should not worry — the proposed change keeps ESC, but adds Enter: so using either of them amount to closing these dialogs.

@ihhub ihhub merged commit 472b88f into ihhub:master Oct 19, 2021
@ihhub
Copy link
Owner

ihhub commented Oct 19, 2021

@a1exsh , thanks for this change! Now it became clearer to me :)

@Branikolog
Copy link
Collaborator

Hi, @ihhub & @a1exsh !
Plus-ultra enhancement! Really missed it. Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement New feature, request or improvement ui UI/GUI related stuff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants