-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Archived" notifications should be referred to as "Read" #3426
Comments
This looks like a good change to me 🙂 |
@sivaraam I agree. It makes more sense to change the name to read |
As far as I remember we used the word "Archived" as this is what the Wikipedia app uses. While building the notifications feature, we tried to follow the same user experience/behavior as the Wikipedia app. Before making changes lets double-check the following:
|
It indeed uses the word "Archived". I find it strange though, given that the website doesn't follow this terminology. Perhaps the website changed it.
Yes, they are. I confirmed this by checking the unread and archived notifications in my account through the app and the website, and they match. |
Thanks for checking both these @6point022. Let us get some more opinions and if everyone agrees then I am okay with this change. |
That's an interesting point. I went searching for the rationale behind using "Archived" in phabricator. Here are some related tasks that I could find:
I don't speak for them but from what I understand from those tasks and from using the app, they've used "Archived" as they provide a flow in which you could treat the list of unread notifications as kind of a backlog of notifications that you would like to get back to. They also provide ways to move a notification "to" and "from" the archive. We could ask them about the rationale, if necessary. That said, I don't think the commons app provides a way to move the notifications "from" the archive to the "unread" group. So, I don't think the commons app is providing the "same user experience/behavior as the Wikipedia app" yet. 😉 As we are show the "Unread" notifications and only provide a way to mark them as read without anyway to mark them as "Unread" like the Wikipedia app, I think it makes sense to do this change. |
@misaochan @macgills I'd really appreciate your inputs to proceed with this issue. |
I say do it |
I'm on board with it. :) |
Summary:
"Read" notifications are referred to as "archived" notifications. However, this is inconsistent with how the website displays the notifications.
What did you expect the app to do, and what did you see instead?
When you open archived notifications, the option to go back says
View unread
which is again, inconsistent. We should either refer to notifications as archived and unarchived, or read and unread. In my opinion, we should follow the latter labelling.Commons app version:
2.12.0-debug-master~d0d663578
Would you like to work on the issue?
Yes, if we can agree the issue is genuine then I'd like to work on this.
The text was updated successfully, but these errors were encountered: