-
Notifications
You must be signed in to change notification settings - Fork 753
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
Haptic Feedback #207
Comments
Will take care of this 👌👌 |
Hello. Any chance to get this PR merged? |
@mikestalker it's been a long time since I made the PR so the branch eventually diverged, I will fix the conflicts this weekend so if it doesn't get merged and you really need my changes, you can just use my fork instead |
Thx @AlbertoSamele. Just switched from NotificationBannerSwift to SwiftMessages and wanted get this haptic effect as it was in old library here as well. Not critical but nice to have for consistency. |
Conflicts solved, my branch is now up to stream if you still want to use it @mikestalker |
Thank you @AlbertoSamele . Hopefully your PR will be merged and released soon. Any chanse for that @wtmoose ? |
@wtmoose can you please merge imrovements from @AlbertoSamele please? |
@wtmoose any chanse to get haptic feedback PR merged? |
Yeah. There were some problematic areas of the PR that prevented me from merging it. I went ahead and made the required updates. You can try it on the head of |
Thanks @AlbertoSamele , thanks @wtmoose |
Maybe it would be a good idea to add some optional haptic feedback to certain alerts.
According to the human guidelines, there's three levels of feedback for notifications: success, warning and error.
At the moment it can be added using event listeners, but adding default support for it on specific configurations would be helpful
The text was updated successfully, but these errors were encountered: