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

Enable applications to report whether they handled an input event #31

Closed
rib opened this issue Sep 19, 2022 · 0 comments · Fixed by #32
Closed

Enable applications to report whether they handled an input event #31

rib opened this issue Sep 19, 2022 · 0 comments · Fixed by #32

Comments

@rib
Copy link
Collaborator

rib commented Sep 19, 2022

When running with NativeActivity / InputQueue then when we "finish" the processing of an event we can tell the input queue if the application handled the event and if not it's possible that there will be some fallback / default handling.

Currently we're reporting that the event was always handled so there will never be any fallback, since this was a simple way to have consistency with GameActivity.

We should make it possible to support fall backs via an InputQueue or potentially support fall backs with GameActivity in the future too.

This was highlighted during the review for this winit backend PR: rust-windowing/winit#2444 where it was also noted that winit may want expose the same notion to allow fall backs for volume controls.

rib added a commit that referenced this issue Sep 19, 2022
The callback given to `AndroidApp::input_events()` is now expected to return
`InputStatus::Handled` or `InputStatus::Unhandled`.

When running with NativeActivity then if we know an input event hasn't been
handled we can notify the InputQueue which may result in fallback
handling.

Although the status is currently ignored with the GameActivity backend.

Fixes: #31
rib added a commit that referenced this issue Sep 19, 2022
The callback given to `AndroidApp::input_events()` is now expected to return
`InputStatus::Handled` or `InputStatus::Unhandled`.

When running with NativeActivity then if we know an input event hasn't been
handled we can notify the InputQueue which may result in fallback
handling.

Although the status is currently ignored with the GameActivity backend.

Since this is a breaking change that also affects the current Winit
backend this updates the winit based examples to stick with the 0.3
release of android-activity for now.

Fixes: #31
@rib rib closed this as completed in #32 Sep 19, 2022
rib added a commit that referenced this issue Sep 19, 2022
The callback given to `AndroidApp::input_events()` is now expected to return
`InputStatus::Handled` or `InputStatus::Unhandled`.

When running with NativeActivity then if we know an input event hasn't been
handled we can notify the InputQueue which may result in fallback
handling.

Although the status is currently ignored with the GameActivity backend.

Since this is a breaking change that also affects the current Winit
backend this updates the winit based examples to stick with the 0.3
release of android-activity for now.

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

Successfully merging a pull request may close this issue.

1 participant