-
-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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] Increase mouse button release timeout and reset it from post_popup
.
#95232
Conversation
Thanks! |
@akien-mga @bruvzg I'm seeing a somewhat frustrating regression from this change (in Windows in my case), where if I open pretty much any popup in Godot (like the top menu) and click something in that popup within a short timeframe it doesn't register the click. Judging by this PR this looks to be deliberate, and lowering this timeout back to 150 ms seems to resolve the issue. I would strongly encourage to at least put this timeout behind an editor setting. I've triggered this dozens of times today alone. I thought it was my mouse acting up at first. |
The timeout should be ignored in a click after releasing the button. |
Timeout should affect only mouse button release, not subsequent clicks. Not sure how it's possible to click without release registering, but I guess we can unset |
Here's what I'm experiencing: godot.windows.editor.dev.x86_64_9WVxMxWGKy.mp4It's tough to show mouse clicks on videos, but I am clicking "Editor Settings..." right after opening the menu, which obviously necessitates releasing the mouse button first, and the click isn't registering. If I wait a short amount of time before clicking it then it registers. |
I'm seeing this with other popups as well, like pretty much any context menu: godot.windows.editor.dev.x86_64_Bl7JGVuLgp.mp4 |
I created an issue for it: #95430 |
Might fix #93265
It seems to be a timing issue. I can't reproduce it with the current settings, but I was able to do it by setting smaller timeout (100 ms). Current timeout is too small for a user interaction and counting from
popup
call (so it includes all window/surface/swapchain creation time, which can be slow).