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

Backends: Android: Added support for io.AddMouseSourceEvent(). #6315

Merged
merged 1 commit into from
Apr 11, 2023

Conversation

PathogenDavid
Copy link
Contributor

This PR adds support for mouse sources to the Android backend similar to f070497 for other backends.

I am marking this as a draft for now as I can't currently test the changes properly.

It turns out the old Android phone I use for testing web stuff in mobile Chrome is far too old to run the example or even be targeted by the current version of Android Studio at all. (It's running KitKat 😅)

Unfortunately the emulator is not really suitable to properly test this change as you can only simulate touch and not pen or mouse input. (I did confirm that it at least properly identifies things as touch in the emulator, but it'd be nice to at least see it identify a Bluetooth mouse properly.)

A generous friend has promised me his old-but-still-modern Android phone, so when I get that in May I'll be able to test this change properly if someone else doesn't volunteer to test it for me before then.

@ocornut ocornut marked this pull request as ready for review April 11, 2023 08:00
@ocornut ocornut merged commit db4c4e3 into ocornut:master Apr 11, 2023
@ocornut
Copy link
Owner

ocornut commented Apr 11, 2023

As someone just asked (#6318) i thought it would be sane to merge, if this compile it very likely works. We can come back to it if we learn anything new.

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

Successfully merging this pull request may close these issues.

2 participants