search

ICIJ / prophecies

An ICIJ app to conduct data validation and cleaning.
https://icij.gitbook.io/prophecies
GNU Affero General Public License v3.0
19 stars 4 forks source link

I mark all notifications as read #82

Closed Soliine closed 3 years ago

Soliine commented 3 years ago

At the moment, the current design of the notification dropdown is the following:

Screenshot 2021-10-04 at 16 49 43

Here is the new design which adds a 'Mark all as read' button (in Figma, you can find it in Task 1 > Comment and notifications):

  1. I have no notifications: Screenshot 2021-10-05 at 17 49 03
  2. I see notification alert: Screenshot 2021-10-05 at 17 49 11
  3. I open the notification dropdown, I hover one of them (please note that as soon as I open the dropdown, the notification alert disappears and become normal again): Screenshot 2021-10-05 at 17 49 59
  4. I hover 'Mark all as read': Screenshot 2021-10-05 at 17 50 10
  5. I clicked on 'Mark all as read': Screenshot 2021-10-05 at 17 50 46

At the moment, we decide to remove all notification alert as soon as the dropdown was once opened, even if the user didn't click on unread notification(s). It allows to not accumulate alerts if there are many of them. If, in the future, users miss notification with this system, we thought of 2 options:

  1. Remove the number of new notifications but keep the ring icon in Danger color
  2. Keep the number but turn the number and the icon in Text color
caro3801 commented 3 years ago

I did not implemented the point 3. "as soon as I open the dropdown, the notification alert disappears and become normal again" because the UI will be inconsistent on page refresh

pirhoo commented 3 years ago

I agree, also this point would add a lot of complexity to have a transitional state on each notification.