Last modified: 2014-08-18 15:31:35 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T63486, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 61486 - Provide linkable anchors to line indicators in diff view
Provide linkable anchors to line indicators in diff view
Status: REOPENED
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
1.23.0
All All
: Low enhancement with 1 vote (vote)
: ---
Assigned To: Nobody - You can work on this!
: design
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-02-18 13:05 UTC by Scott Martin (http://enwp.org/user:scott)
Modified: 2014-08-18 15:31 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Scott Martin (http://enwp.org/user:scott) 2014-02-18 13:05:37 UTC
It would be of use to be able to link to a specific change of any given diff, especially in the case of a complex diff. For example, in this diff between two versions of the article "Wikipedia" on the English Wikipedia that I selected at random:

https://en.wikipedia.org/w/index.php?title=Wikipedia&diff=594765035&oldid=590925507

the sentence "Also, all pie-charts may be displayed properly in desktop view, but not in mobile view." has been added to the article at what was previously line 449. If I want to direct someone to that particular change, I need to instruct them to either scroll down to the "Line 449" indicator or search for the text in question. This is inefficient.

So, I suggest that every change in the diff (displayed with a "+" or "-" indicator) is accompanied with an anchor link (preferably using an icon of the chain type commonly used online for permanent links). These would be simply calculated as integers starting at 1.

Let's pretend that in the case of the example above, the change I refer to was the 56th in the diff (it wasn't - actually counting what it was would take a long time). An anchor link for that change would generate the following URL:

https://en.wikipedia.org/w/index.php?title=Wikipedia&diff=594765035&oldid=590925507#56
Comment 1 PiRSquared17 2014-02-23 05:55:45 UTC
Would this disrupt the UI? Should it be enabled for all users? Would the anchor be to the left, right, above, or below the +/- ?
Comment 2 PiRSquared17 2014-02-23 05:56:45 UTC
What if we just make the + and - symbols into links (and add hover text)?
Comment 3 Scott Martin (http://enwp.org/user:scott) 2014-02-23 22:43:10 UTC
That could work. I do quite like having a definite visible icon for discoverability, but some people might see that as clutter. If there were, I would suggest that they be slightly lower contrast until mouseover, to reduce visual noise. Would be interesting to hear from design people about this.
Comment 4 PiRSquared17 2014-02-23 23:27:19 UTC
This might break links if the diffing algorithm is changed significantly, although I do like this idea in general.
Comment 5 Scott Martin (http://enwp.org/user:scott) 2014-02-24 11:57:26 UTC
Rats, that's a really good point. Well, I can think of a way to avoid breakage - if the anchor URL specifies what kind of diff is in use, and that particular diff algorithms are always kept available. 

In that case, my example URL from above would look something like the following (pretending that our current diff algo is called "diff1"):

https://en.wikipedia.org/w/index.php?title=Wikipedia&diff=594765035&oldid=590925507&difftype=diff1#56

A diff without difftype= would always use the latest algorithm, whatever it may be. Obviously this raises the complexity somewhat; I don't know what the internal policy with regards to diff algorithms is. I would like to imagine that access could still be provided to older ones in the event of changes. Someone who's actually a developer, unlike me, can hopefully shed some light on that.
Comment 6 PiRSquared17 2014-02-25 16:01:55 UTC
(re to comment 5)
Sounds like a reasonable idea, but I'm not sure if the devs would like it.
Comment 8 Ricordisamoa 2014-04-25 14:23:30 UTC
Alternative proposal: bug 63707
Comment 9 Andre Klapper 2014-08-16 10:13:09 UTC
Hi Scott. Thanks for taking the time to report this!
This particular problem has already been reported into our bug tracking system, but please feel free to report any further issues you find.

*** This bug has been marked as a duplicate of bug 2313 ***
Comment 10 Ricordisamoa 2014-08-16 11:57:04 UTC
(In reply to Andre Klapper from comment #9)

AFAIU, bug 2313 and bug 63707 are about linking line indicators to related paragraphs in the content view, while this bug is about anchors to line indicators themselves.
Comment 11 Scott Martin (http://enwp.org/user:scott) 2014-08-18 11:46:07 UTC
Comment 10 is correct. This bug is not a duplicate of bug 2313, so I'm unsetting that.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links