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

Fix mouse_in_window state with confined mouse #85872

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Sauermann
Copy link
Contributor

_update_mouse_over is called whenever the mouse is over the Window. So mouse_in_window needs to be set in all cases.

resolve #85713
regression from #80334

`_update_mouse_over` is called whenever the mouse is over the Window.
So `mouse_in_window` needs to be set in all cases.
@Sauermann Sauermann added bug topic:core topic:input regression cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release labels Dec 6, 2023
@Sauermann Sauermann added this to the 4.3 milestone Dec 6, 2023
@Sauermann Sauermann requested a review from a team as a code owner December 6, 2023 23:02
akien-mga
akien-mga previously approved these changes Dec 7, 2023
@Sauermann Sauermann marked this pull request as draft December 7, 2023 07:43
@akien-mga
Copy link
Member

Are there still outstanding issues with this PR, or was it just put back in draft for some further testing?

@Sauermann
Copy link
Contributor Author

I put the PR into draft status, because during my tests I found a bug. A more elaborate solution would be necessary.

@Sauermann Sauermann modified the milestones: 4.3, 4.x Mar 7, 2024
@akien-mga akien-mga dismissed their stale review March 8, 2024 12:03

Needs more work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cherrypick:4.2 Considered for cherry-picking into a future 4.2.x release regression topic:core topic:input
Projects
None yet
2 participants