carverauto / flutter

🚨 ChaseApp opensource mobile app - follow live police chases 🚔
https://chaseapp.tv
MIT License
3 stars 0 forks source link

Notifications - user feedback #143

Closed mfreeman451 closed 2 years ago

mfreeman451 commented 2 years ago

Some of the feedback that has come back on the notification system is that it needs to be improved.

  1. In the main screen, the notification "Bell" icon doesn't do anything, it should be made 'bold' when it is enabled' and light gray when it is disabled or have a slash through it
  2. In settings, users should be able to choose what notifications are enabled and disabled. Users are complaining they can't disable notifications.
rutvik110 commented 2 years ago

On the second point there, I think we'll have to make it more clear that some are default and others are optional notification streams. May be add a little header above default ones that says they can't be turned off!

I had added something for the first point, but I think I'll need to improve it.

rutvik110 commented 2 years ago

Updates on 1 point! https://user-images.githubusercontent.com/65209850/156217628-8b6a27bd-4cb5-4b3d-afa0-ca7f40de5e3b.mp4

All types of notification streams settings are editable now.

rutvik110 commented 2 years ago

I believe with the latest updates we have resolved this issue. So closing for now :)