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

iGUI in character creation broken #77636

Closed
Mer1ck opened this issue Nov 7, 2024 · 4 comments · Fixed by #77651
Closed

iGUI in character creation broken #77636

Mer1ck opened this issue Nov 7, 2024 · 4 comments · Fixed by #77651
Labels
<Bug> This needs to be fixed Info / User Interface Game - player communication, menus, etc. (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Mer1ck
Copy link
Contributor

Mer1ck commented Nov 7, 2024

Describe the bug

The GUI when you attempt to exit character creation does not function. It appears when attempting to exit or close the CDDA window, and the Yes/No options cannot be selected. I've tried using keyboard, mouse, and shortcuts, and am unable to select "yes" or "no" on the popup.

Attach save file

N/A

Steps to reproduce

Create World
Start to create Custom Character
Attempt to exit via closing the window or Alt + F4
Bug should occur

Expected behavior

You should be able to select either option and leave character creation/quit the game

Screenshots

No response

Versions and configuration

OS: Windows 11
Game Version: 19807c7
Graphics: Tiles

Additional context

No response

@Mer1ck Mer1ck added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Nov 7, 2024
@Night-Pryanik Night-Pryanik added <Bug> This needs to be fixed Info / User Interface Game - player communication, menus, etc. (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Nov 7, 2024
@ZhilkinSerg
Copy link
Contributor

ZhilkinSerg commented Dec 27, 2024

@andrei8l While we can consider the original issue resolved, Alt+F4 is not handled properly now (at least on Windows):

  1. Pressing Alt+F4 in character creation menu shows Y/N prompt:
    • pressing Y closes game window instead of returning back to main menu.
    • pressing N returns back to character creation. Then if you then press ESC to come back to main menu - you receive another Y/N prompt and if you press Y game window would be closed.
  1. Pressing ESC in character creation menu (when not pressing Alt+F4 at any time beforehand) shows Y/N prompt:
    • pressing Y returns back to main menu as expected;
    • pressing N returns back to character creation menu as expected.

Similar wrong behavior is in other menus, e.g. if you open any popup in Settings menu (say, default name) and use Alt+F4 there, nothing happens at first, but if you then press ESC to close popup and then hit ESC again to close Settings menu and return back to main menu, game window would be closed instead.

It looks like exit signal is pending and handled once current ui loop is finished.

P.S.: In build cdda-experimental-2024-10-21-1323 and earlier builds, pressing Alt+F4 made application exit immediately once Alt+F4 is pressed.

Build cdda-experimental-2024-10-21-1656 introduced changes from #75999 and is the first one that has the issue with non-immediate handling of Alt+F4.

@andrei8l
Copy link
Contributor

I still maintain that we should revert #67893 and all of my follow up PRs that start with #75999. #67893 is a leech issue and I don't want to spend any more time on it, sorry.

@RenechCDDA
Copy link
Member

Similar wrong behavior is in other menus, e.g. if you open any popup in Settings menu (say, default name) and use Alt+F4 there, nothing happens at first, but if you then press ESC to close popup and then hit ESC again to close Settings menu and return back to main menu, game window would be closed instead.

Which settings menu is this? I can't reproduce.

@ZhilkinSerg
Copy link
Contributor

One of these:

image
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bug> This needs to be fixed Info / User Interface Game - player communication, menus, etc. (S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants