Missing Mark all as Read for Notifications

Considering the Profile “landing” page is slated to become the Summary soon and this is somewhat edge case, probably not worth worrying about it much.

Typically, unless a Notifications link is specifically clicked I land on Profile Activity.

If I go to Profile (Activity) and click Notifications, there is a “Mark all as Read” button.

But if I go to User Card -> Avatar -> Profile (Activity) and click Notifications, the button is not there until I do a page reload.

「いいね!」 6

On imgur they were saying it sometimes appears and sometimes does not – very likely a bug of some kind.

「いいね!」 2

I thought @sam removed the Mark all as read button a while back…I can’t find that button at all here on Meta.

It’s here https://meta.discourse.org/my/notifications

Where?

:smiley:

「いいね!」 1

I could see it if I visited my notifications before visiting this topic. If I go back now it is gone…

It’s only visible when you have unread notifications.

https://github.com/discourse/discourse/blob/514c3976f05785f487d275af1e96c43533e7a478/app/assets/javascripts/discourse/controllers/user-notifications.js.es6#L8-L8

「いいね!」 5

Vanishing buttons and disabled buttons seem to consistently cause confusion for no gain, we should probably change this

「いいね!」 10

I support leaving it there for notifications since this has come up a few times already.

「いいね!」 4

:bomb:

https://github.com/discourse/discourse/commit/8df69dba83d733038dfb2afe46146875538a5f3a

「いいね!」 6