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

PopupMenu scrolling is unusable with keyboard/controller input #80224

Closed
rsubtil opened this issue Aug 3, 2023 · 0 comments · Fixed by #80271
Closed

PopupMenu scrolling is unusable with keyboard/controller input #80224

rsubtil opened this issue Aug 3, 2023 · 0 comments · Fixed by #80271

Comments

@rsubtil
Copy link
Contributor

rsubtil commented Aug 3, 2023

Godot version

4.2.dev (latest)

System information

Arch Linux

Issue description

Selecting items on PopupMenus by using a keyboard/controller is unusable due to a visual bug. The scrolling position jumps constantly back and forth:

Peek.2023-08-03.15-59.mp4

The selection itself is working properly; only the logic for scrolling to the selected item is wrong. I've bisected it to #57692 (cc @YeldhamDev), and restoring the scroll_to_item code to before this PR fixes the issue:
https://github.com/godotengine/godot/pull/57692/files#diff-7ee2fe488be40211cc85edff72e3668728514546ee530987099f7dc1f057f85cR1294-R1306

Steps to reproduce

  • Have an OptionButton with enough options to make it scrollable.
  • Attempt to scroll by holding the up/down keys to traverse items.
  • Items out of the initial selection will make the scroll position jump back and forth.

Minimal reproduction project

Repro.zip

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants