sumanah wrote:

Lejonel, can you still reproduce this?

Steven and Oliver, would you agree that this is a high-priority bug?

I'd put it as normal; it's somewhat disruptive to a specific way of working, but it isn't going to break anything and that workflow is in itself relatively new and still getting established. Marking as such now.

(In reply to comment #1)

can you still reproduce this?

Viewing an old revision still removes "Updated since last visit".
But viewing diffs between old revisions seems to have been fixed, it does not remove update markers.

IMHO viewing an old revision (both as a diff and as an old version of that page) should update the markers up to that point. So currently when viewing an old version too many of these markers are removed (i.e. all), viewing a diff removes too few (i.e. none).

I was going to file a bug saying that viewing old revisions (and diffs from revisions you've already seen to revisions you hadn't yet) should update the markers and I've found this. Rephrasing the bug summary a little.

Change 84315 had a related patch set uploaded by Bartosz Dziewoński:
Correctly update wl_notificationtimestamp when viewing old revisions

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

Change 84315 merged by jenkins-bot:
Correctly update wl_notificationtimestamp when viewing old revisions

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

Hooray, merged!

The patch touched a lot of different things in the codebase, so I'm not going to backport it to 1.23 just in case it accidentally broke one of them. Adjusting milestone accordingly.

(I meant to 1.22, of course.)