search

dennisbruner / vue-native-notification

Vue.js plugin for native notifications
https://www.npmjs.com/package/vue-native-notification
MIT License
71 stars 13 forks source link

Not supported on Safari iOS 14 #17

Open dv336699 opened 3 years ago

dv336699 commented 3 years ago

Hi, this line will not work and cause an error on Safari iOS 14

https://github.com/dennisbruner/vue-native-notification/blob/7773d030c8098cb08ff75568a3cfd8c6960fb2f7/index.js#L41

There are also a few others https://caniuse.com/?search=notifications

I'll try to send a PR soon.

maximerenou commented 3 years ago

That's right. It crashes my app, so nothing displays on Safari (iOS).

Adding try / catch around requestPermission() fixes the problem.