Page MenuHomePhabricator

[mobile] the counter for seen Alerts/Notices is not displayed correctly
Open, Needs TriagePublic

Assigned To
None
Authored By
Etonkovidova
Jan 20 2018, 12:29 AM
Referenced Files
F60729023: notifications_circle.gif
May 27 2025, 11:55 PM
F58968897: image.png
Apr 3 2025, 12:39 AM
F34566561: Screen Shot 2021-07-26 at 5.59.47 PM.png
Jul 27 2021, 1:22 AM
F34566539: Screen Shot 2021-07-26 at 5.56.44 PM.png
Jul 27 2021, 1:22 AM
F34566531: Screen Shot 2021-07-26 at 5.45.19 PM.png
Jul 27 2021, 1:22 AM
F34566564: Screen Shot 2021-07-26 at 6.00.02 PM.png
Jul 27 2021, 1:22 AM
F34566533: Screen Shot 2021-07-26 at 5.44.29 PM.png
Jul 27 2021, 1:22 AM
F34566568: Screen Shot 2021-07-26 at 6.00.29 PM.png
Jul 27 2021, 1:22 AM

Description

The behavior of displaying seen/unseen notifications should be consistent with the desktop to provide users with the same logic for dealing with notifications. Presently, the counter for seen notifications behaves incorrectly.

(1) Only seen notifications are present. The notification that are seen, should be displayed with a number in grey color. On mobile there will be no indication to a user that there are still notifications that should be marked as read.
Teted on testwiki wmf.15

desktopmobile
nine notifications are seen but not read
Screen Shot 2021-07-26 at 5.45.19 PM.png (580×1 px, 98 KB)
Screen Shot 2021-07-26 at 5.44.29 PM.png (1×754 px, 192 KB)
Screen Shot 2021-07-26 at 5.56.44 PM.png (1×770 px, 234 KB)

(2) unseen notifications

  • the correct number of not-seen notifications is displayed in red

Screen Shot 2021-07-26 at 5.59.47 PM.png (1×766 px, 236 KB)

  • opening the notifications' flyout

Screen Shot 2021-07-26 at 6.00.02 PM.png (1×760 px, 132 KB)

  • after the notifications' status changed to seen (the flyout was opened), the number 1 is displayed in grey - it should be 7. F34566566
  • (updated 2025) after the notifications' status changed to seen (the flyout was opened), the number changes to 0, it is still displayed in red.

image.png (1×527 px, 37 KB)

  • a user navigates to a different article, the counter disappears.

Screen Shot 2021-07-26 at 6.00.29 PM.png (1×758 px, 236 KB)

Event Timeline

Etonkovidova renamed this task from Minerva skin: the icon and the counter for seen Alerts/Notices will not be displayed. to [mobile] the counter for seen Alerts/Notices is not displayed correctly.Jul 27 2021, 1:22 AM
Etonkovidova updated the task description. (Show Details)
Etonkovidova added a subscriber: Jdlrobson.

This may have been a bug once, but during the work on T342907, it has been promoted to a feature.

The number of seen notifications is hidden by this line: https://gerrit.wikimedia.org/g/mediawiki/skins/MinervaNeue/+/f0cf4a47fc3f1af98bc2c232778c7fbf35a8a102/resources/skins.minerva.loggedin.styles/styles.less#13

This may have been a bug once, but during the work on T342907, it has been promoted to a feature.

The number of seen notifications is hidden by this line: https://gerrit.wikimedia.org/g/mediawiki/skins/MinervaNeue/+/f0cf4a47fc3f1af98bc2c232778c7fbf35a8a102/resources/skins.minerva.loggedin.styles/styles.less#13

Can we decline this ticket then or close it as invalid?

Well… I think it'd be better to show the counter. But it's up to y'all to declare that this is expected 🤷‍♂️

Well… I think it'd be better to show the counter. But it's up to y'all to declare that this is expected 🤷‍♂️

Agree. Maybe some design review is needed? There are two small issues (which might be considered features)

  • displaying 0 in a red circle. The red color implies a call to an action, but what actions zero notifications might require?
  • changing notifications icons - from a circle (un-seen/seen notifications) to a bell (read notifications)

Previously the red circle became gray and then on page refresh became a bell.

Previously the red circle became gray and then on page refresh became a bell.

Yes, I used to see it too - the grey circle was for seen notifications. I re-checked on testwiki wmf.3 - for seen notification the red circle with zero counter is consistently displayed (see the animated gif below). After seeing notification and marking them all as read, the notification icon will stay as the red circle with 0. Reloading a page with zero red circle changes the notification icon to the bell

notifications_circle.gif (784×375 px, 843 KB)