-
-
Notifications
You must be signed in to change notification settings - Fork 373
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
Android vibration fixes. #1804
Android vibration fixes. #1804
Conversation
MAJigsaw77
commented
Jun 16, 2024
•
edited
Loading
edited
- Fixed some deprecation warnings.
- Cancel the vibration when the app is being paused or destroyed.
@player-03 Can i get a review on this pr? |
templates/android/template/app/src/main/java/org/haxe/lime/GameActivity.java
Outdated
Show resolved
Hide resolved
I'd rather duplicate the SDK version check, not the math.
Interesting. According to the documentation, the first entry in the array is the delay before vibration starts, so they recommend leaving that 0. The question is, is this delay a bug or a feature? If it's a bug, we could fix it now. If it's a feature, we probably want to add an argument to toggle it on or off. |
`GameActivity` doesn't print any other messages, so this seems out of place. Plus, it would happen every time even if there's no vibration ongoing.
I don't really know what to say tbh |