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

Focus "outline" for sliders moves with scrolling #97825

Closed
ghost opened this issue Oct 4, 2024 · 0 comments · Fixed by #97946
Closed

Focus "outline" for sliders moves with scrolling #97825

ghost opened this issue Oct 4, 2024 · 0 comments · Fixed by #97946

Comments

@ghost
Copy link

ghost commented Oct 4, 2024

Tested versions

Reproducible: v4.0.stable.official [92bee43], v4.4.dev.flathub [97ef3c8]
Not reproducible: v3.6.stable.official [de2f0f1]

System information

Godot v4.4.dev (97ef3c8) - Freedesktop SDK 23.08 (Flatpak runtime) - Wayland - Vulkan (Forward+) - dedicated AMD Radeon RX 570 Series (RADV POLARIS10) - Intel(R) Core(TM) i3-10100F CPU @ 3.60GHz (8 Threads)

Issue description

Slider focus "indicator" will move with scrolling when it shouldn't.
Yeah, not the greatest explanation, just watch the video:

2024-10-04.23-10-16.webm

I didn't tested using in-game nodes, but this consistently happens thought editor, for example, settings window, node inspector, etc.
I tested in both 4.0 and 4.4 versions and it happens there, but not in 3.6:

3_6.webm

So something changed in 4.0 and onward after that.

Steps to reproduce

  1. Pick scrollbar with cursor pointer (for example, editor settings window or node inspector) and don't release it.
  2. Move cursor to any slider ui element (for example, custom ui scale in editor settings)
  3. Move cursor up/down and observe how focus "outline" start moving with scrolling (it will disappear when cursor will leave slider "hitbox")

Minimal reproduction project (MRP)

This can be reproduced in Godot editor itself.

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.

2 participants