Hide TreeItem edit popup on scroll.#112998
Conversation
|
Found an issue with this implementation, If you edit then scroll while hovering over the scroll bar (not the tree) then the tree doesn't pickup the scroll event and thus the same issue as before occurs again. Will fix this in the next commit and will only stop editing if the scroll value actually changed. |
|
Fixed both issues. Now it will capture the input whether or not anything is focused since it checks directly if the scroll value changed. And now if the scroll value didint change then it wont stop editing. 2025-11-22.17-18-03.mp4 |
KoBeWi
left a comment
There was a problem hiding this comment.
I think the more expected behavior would be to move the popup with scroll, but this works too. It cancels the editing, but it's not something that can happen by accident (I think?), so it shouldn't cause problems.
202946d to
cb78af7
Compare
|
Thanks! And congrats for your first merged Godot contribution 🎉 |
…p-on-wheel Hide TreeItem edit popup on scroll.
Before, you could scroll while editing an item on a Tree node.
Before:
before.mp4
After:
after.mp4