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

imGUI: regression in not being able to switch options with capital letters #73058

Closed
Karol1223 opened this issue Apr 16, 2024 · 9 comments
Closed
Labels
<Bug> This needs to be fixed ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds (S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Karol1223
Copy link
Contributor

Describe the bug

Previously, when closing the game you could press Y to switch to Yes and N to switch to No. This is no longer possible.

Attach save file

Vanilla-trimmed.tar.gz

Steps to reproduce

  1. Try to Save and Quit
  2. Press Y to try and switch the chosen option to Yes
  3. Fail
  4. Use arrow keys like a madman

Expected behavior

I don't expect regressions. I mean, I do, but I'd like to see it fixed anyway.

Screenshots

image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.22631.3447 (23H2)
  • Game Version: 0.G-8844-g14fd9eaaf9-dirty [64-bit]
  • Graphics Version: Tiles
  • Game Language: English [en]
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

No response

@Karol1223 Karol1223 added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Apr 16, 2024
@RenechCDDA RenechCDDA added the ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds label Apr 16, 2024
@esotericist
Copy link
Contributor

to elaborate a little on this, in settings/interface/accessibility, there's a setting:
"force capital/modified letters in prompts":
image

if a choice is being made to drop this functionality, it should be explicit, and we shouldn't have a setting present that claims that functionality.

(i personally would like to keep it, though)

@esotericist esotericist added (S2 - Confirmed) Bug that's been confirmed to exist <Bug> This needs to be fixed and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Apr 30, 2024
@Karol1223 Karol1223 changed the title imgUI: regression in not being able to switch options with letters imGUI: regression in not being able to switch options with capital letters May 4, 2024
@Brambor
Copy link
Contributor

Brambor commented May 8, 2024

I understand this to be solved by #73420

@Brambor Brambor closed this as completed May 8, 2024
@Karol1223
Copy link
Contributor Author

It is not done. The PR was done due to a misunderstanding between what this issue highlights and solves an entirely unrelated bug.

@GuardianDll GuardianDll reopened this May 8, 2024
@katemonster33
Copy link
Contributor

since @esotericist 's view of the issue was wrong then I no longer believe this is confirmed. I tried a fresh experimental build from today and compared it to 0.G. On either build I:

  1. Load game
  2. Hit escape
  3. Hit 'S' (shift + s)
  4. Hit 'Y' (shift + y)

and verify that I am sent back to the main menu. I am using the config you posted. Can't reproduce.

@Karol1223
Copy link
Contributor Author

Karol1223 commented May 9, 2024

I tried it on the newest experimental as well when confirming the PR didn't fix it and it still happens to me.

Going through the steps does nothing. Only using arrow keys after pressing shift+Y actually closes it. (And I mean pressing the arrow keys without using Enter, this is weird)

...admittedly, though, I may be stupid. Brambor opened an issue about the selected options not being highlighted (#73583). It's entirely possible the keybinds work but I didn't realize that it was highlighting that was broken. Apparently that's invalid? If so, nevermind.

@Brambor
Copy link
Contributor

Brambor commented May 11, 2024

Apparently that's invalid? If so, nevermind.

I marked it as invalid since it is not a master problem. It is a problem after applying some changes.

@Karol1223
Copy link
Contributor Author

This is getting progressively weirder and weirder. I just made a new save and... this bug no longer occurs? But it did occur on my old save on the same exact version yesterday...

@Brambor
Copy link
Contributor

Brambor commented May 11, 2024

Could something be saved through uistate.h? You could try looking at the save files.

Very wild guess, probably not.

@Karol1223
Copy link
Contributor Author

Could something be saved through uistate.h? You could try looking at the save files.

Very wild guess, probably not.

I tested it on that same savefile that wasn't working and... now it's working? I didn't update the game inbetween. As far as I can see nothing changed except me turning on NumPad? But after turning NumPad back off it was working still.

I guess the issue is resolved? No idea how, though...

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 ImGui Anything related to the new ImGui UI for SDL/tiles or ImTui for curses builds (S2 - Confirmed) Bug that's been confirmed to exist
Projects
Status: Done
Development

No branches or pull requests

6 participants