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

PassThroughInputManager applies mouse release input without providing source context (i.e. whether it comes from a pen) #6508

Open
frenzibyte opened this issue Jan 24, 2025 · 0 comments

Comments

@frenzibyte
Copy link
Member

This makes the e.CurrentState.Mouse.LastSource state incorrect in mouse up events. I really think it's not a good idea that we are mixing pen input into mouse input instead of separating it into its own PenState and synthesize mouse input exactly like how touch works.

If there's anything bad with that approach, it is the fact that pen-specific events will be propagated throughout the hierarchy first, and if a single class handles it, nothing above will be aware of anything.

Requires further exploration to avoid the issue above and make the input flow work more by iterating through each Drawable in the input queue and first triggering the pen-specific event, then fall back to the mouse-specific event if the pen one was not handled. Or perhaps something much more sophisticated than that.

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

No branches or pull requests

1 participant