Page MenuHomePhabricator

Add diff link to "You have new messages"
Closed, ResolvedPublic

Description

Author: artslave

Description:
A small, unobtrusive "diff" link added to the orange "You have new messages" box
(perhaps in the lower right corner) would make it a little easier for users to keep up
with their talk pages, especially when they have multiple conversations going on.

Very helpful for experienced and active users with long busy talk pages, and not too
confusing for newbies, one would hope....

Just an idea - discard if it's expensive for any reason.


Version: 1.6.x
Severity: enhancement

Details

Reference
bz1572
TitleReferenceAuthorSource BranchDest Branch
maintain-kubeusers: Bump memory for wd-shex-inferrepos/cloud/toolforge/toolforge-deploy!218taavitaavi/wd-shex-infermain
maintain-kubeusers: bump to 0.0.121-20240219092902-759465a7repos/cloud/toolforge/toolforge-deploy!199project_1317_bot_df3177307bed93c3f34e421e26c86e38bump_maintain-kubeusersmain
wd-shex-infer: update also the limitrangerepos/cloud/toolforge/toolforge-deploy!198dcaroincrease_quota_T357209main
maintain-kubeusers: increase quota for wd-shex-inferrepos/cloud/toolforge/toolforge-deploy!197dcaroincrease_quota_T357209main
runner: Fix eslint commit message generationrepos/ci-tools/libup!20taavitaavi/messagemaster
runner: Deal with nested eslint config filesrepos/ci-tools/libup!18taavitaavi/eslint-subconfigmaster
Customize query in GitLab

Revisions and Commits

Event Timeline

bzimport raised the priority of this task from to Medium.Nov 21 2014, 8:12 PM
bzimport set Reference to bz1572.

The message _is_ linked with the User_talk page.

What do you mean exactly, pls. can you elaborate ?

Tom

artslave wrote:

(In reply to comment #1)

The message _is_ linked with the User_talk page.
What do you mean exactly, pls. can you elaborate ?
Tom

I mean a "diff" link -- so that one could go directly to a diff between the current and previous version of your talk page,
highlighting the new addition.

Often when I'm carrying on multiple conversations on different parts of the page, I click the "new messages" link to go to
User_Talk, then the "history" tab, then the "last" link to find what has changed -- rather than scrolling through several
complex sets of indented messages trying to spot the new addition (and sometimes missing additional comments elsewhere).

I'm guessing it might be an unnecessary drain on the server, and the answer is "keep your talk page better trimmed", but
it /would/ be an improvement on the talk system if there was a simple way for "you have new messages" to lead you directly to
those new messages, instead of just dumping you into your inbox with no indication of what's actually new.

Just throwing it out there in case there's a simple, inexpensive solution.

(In reply to comment #2)

I mean a "diff" link -- so that one could go directly to a diff between the

current and previous version of your talk page,

highlighting the new addition.

oops, I did not read carefully exact, but now I fully understand.

I program for you a

(diff) link

AND (diff-to-last-visited revision) link !!

It comes for free with my ENotif very soon
http://bugzilla.wikipedia.org/show_bug.cgi?id=454 ;-))
Thanks for the suggestion.

Tom

depending on ENotif because of timestamp / revision storages (last seen version)

Catherine:

your idea with the "diff" links *is* implemented in ENotif/EConfirm v3.16, see
http://bugzilla.wikipedia.org/show_bug.cgi?id=454 .
I cannot say, whether this will be implemented in public release version or not,
but I fully support it.

  • Bug 3219 has been marked as a duplicate of this bug. ***

implemented in MediaWiki special version EnotifWiki http://www.enotifwiki.org
(which runs this code live). See also screenshot of the modied marker on
http://meta.wikimedia.org/wiki/Enotif#Features

robchur wrote:

Patch against HEAD

Dah-dum! Patch against HEAD.

attachment new-talk-diff-link.patch ignored as obsolete

see http://www.enotifwiki.org (your patch does not work - the difference to the
last seen version of the watching(!) user is meant. It needs a new column in
table watchlist)

robchur wrote:

(In reply to comment #10)

see http://www.enotifwiki.org (your patch does not work - the difference to the
last seen version of the watching(!) user is meant. It needs a new column in
table watchlist)

Unless I'm mistaken, the request was for a link to the diff of the most recent
change to the user's talk page. That is what I've implemented.

(In reply to comment #11)

(In reply to comment #10)

see http://www.enotifwiki.org (your patch does not work - the difference to the
last seen version of the watching(!) user is meant. It needs a new column in
table watchlist)

Unless I'm mistaken, the request was for a link to the diff of the most recent
change to the user's talk page. That is what I've implemented.

ok. I see what you mean (I had filed another bugzilla some months ago, where I
suggested to change "diff" to "prev" to indicate, that a difference to the
previous version is meant; however, this suggestion was dismissed, because it
was a major change to the user interface).

Sorry. You are right. _I_ misunderstood, because I was biased to _my_
understanding of the word "diff".

(In reply to comment #13)

No problem.

Rob, it would be a pleasure to me, if you "play" with http://www.enotifwiki.org
to see, how the new "new message marker" works. I mean: _really_ guiding any
user to the difference between the current and the last-seen-version. Take your
time and let me know your thoughts about it.

avarab wrote:

Comment on attachment 1221
Patch against HEAD

Patch had input validation issues and didn't apply.

avarab wrote:

RESOLVED in CVS HEAD, now shows a "diff to penultimate revision" link.

epriestley added a commit: Unknown Object (Diffusion Commit).Mar 4 2015, 8:22 AM
epriestley added a commit: Unknown Object (Diffusion Commit).
Diffusion added a commit: Unknown Object (Diffusion Commit).Mar 4 2015, 8:22 AM
Diffusion added a commit: Unknown Object (Diffusion Commit).