Page MenuHomePhabricator

Add "mark as read" as a deprioitized secondary action to every unread notification
Closed, ResolvedPublic

Description

If I recall correctly, @Pginer-WMF's designs called for "mark as read" to be an option in the dotdotdot menu, but right now it doesn't appear there.

mark-as-read.png (279×1 px, 35 KB)

Regarding the icons:

  • For mark as read: we are using the "check" icon, a tick mark to indicate completion.
  • For mark as unread we are using the "sun" icon, basically a bright spot to indicate the notification is new again:

Event Timeline

Catrope raised the priority of this task from to Medium.
Catrope updated the task description. (Show Details)
Catrope added subscribers: Catrope, Mooeypoo, Pginer-WMF.

Assigning to Pau for clarification and icon assets.

Initially we were a bit hesitant to add the "mark as read" action because it was a bit redundant, but from what we have seen in the testing sessions it seems it can be convenient to anticipate that there will be a "mark as unread" once the notifications became read. So I think it make sense to add it.

I've illustrated both marking as read and unread in a mockup below for further details:

mark-as-read.png (279×1 px, 35 KB)

Regarding the icons:

  • For mark as read: we are using the "check" icon, a tick mark to indicate completion.
  • For mark as unread we are using the "sun" icon, basically a bright spot to indicate the notification is new again:
Pginer-WMF updated the task description. (Show Details)
Pginer-WMF set Security to None.

Change 265549 had a related patch set uploaded (by Mooeypoo):
Add 'mark as read' as secondary action to unread notifications

https://gerrit.wikimedia.org/r/265549

Just a note: At the moment, secondary actions do not have 2 lines (title + description) so the commit that I submitted only has "mark as read" without the second explanation line.

If we want to add the functionality of second line / descriptions for these links, we should have it as another ticket and discuss the full behavior -- one question, for instance, is whether all secondary actions will have an explanation line or not? Would thinks look different between having one and not having one?

Just a note: At the moment, secondary actions do not have 2 lines (title + description) so the commit that I submitted only has "mark as read" without the second explanation line.

If we want to add the functionality of second line / descriptions for these links, we should have it as another ticket and discuss the full behavior -- one question, for instance, is whether all secondary actions will have an explanation line or not? Would thinks look different between having one and not having one?

We already have infra for this in the presentation models, it's the 'description' field in the secondary links array. I believe it's empty for all existing secondary links. We'd just need it to be rendered by the frontend code.

Change 265549 merged by jenkins-bot:
Add 'mark as read' as secondary action to unread notifications

https://gerrit.wikimedia.org/r/265549

Note for re-testing:

  • check betalabs when it'll be available
  • check testwiki (see below)

In testwiki, 'Mark as read' is not present and the design is different:

Screen Shot 2016-01-28 at 2.25.50 PM.png (432×657 px, 65 KB)

Re-checked in betalabs - clicking on 'Mark as read' marks the message as read. Other links do not work - T125160: cross-wiki notifications: the secondary action links - User page and Flow board - do not work.

Screen Shot 2016-01-28 at 2.46.33 PM.png (283×714 px, 51 KB)

Marking this as resolved. @Pginer-WMF, please note that:

  • the styling of all elements in the fly-out menu looks pretty different than in your designs.
  • we can use secondary text to describe these options if we need it (Roan says there is the facility), but will need consistent language and design.

Is there a separate ticket for these? If not, do you want to create one?

Marking this as resolved. @Pginer-WMF, please note that:

  • the styling of all elements in the fly-out menu looks pretty different than in your designs.
  • we can use secondary text to describe these options if we need it (Roan says there is the facility), but will need consistent language and design.

Is there a separate ticket for these? If not, do you want to create one?

We have T126734: Items in the more actions menu of notifications are inconsistent which is part of the pending styling adjustments I compiled at T119374 (consider adding there any missing adjustment as a sub-task)