Last modified: 2006-05-30 22:17: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 T8125, the corresponding Phabricator task for complete and up-to-date bug report information.
First Last Prev Next    This bug is not in your last search results.
Bug 6125 - Add links to edit old versions to diff views
Add links to edit old versions to diff views
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
1.7.x
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency tree / graph
 
Reported: 2006-05-28 22:01 UTC by Ilmari Karonen
Modified: 2006-05-30 22:17 UTC (History)
0 users

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

Attachments
Patch to add edit links to diff header (3.06 KB, patch)
2006-05-28 22:04 UTC, Ilmari Karonen 		Details
Alternative patch, using a new "editold" message instead of "editsection" (35.86 KB, patch)
2006-05-28 22:17 UTC, Ilmari Karonen 		Details
Add an attachment (proposed patch, testcase, etc.)

Description Ilmari Karonen 2006-05-28 22:01:04 UTC 
For convenience in reverting vandalism, diff pages should have direct links to
edit either of the displayed revisions.  A JavaScript hack to accomplish this
has been proposed on the English Wikipedia
(http://en.wikipedia.org/wiki/MediaWiki_talk:Monobook.js#Request:_Edit_L.2FR_for_diffs),
but it would in my opinion be preferable to implement this directly in
DifferenceEngine.php.

See also http://bugzilla.wikimedia.org/show_bug.cgi?id=1783

A patch follows.
Comment 1 Ilmari Karonen 2006-05-28 22:04:12 UTC 
Created attachment 1853 [details]
Patch to add edit links to diff header

The patch currently uses the "editsection" message for the text of the edit
link.  A quick grep of the Messages*.php files suggests that this message is
appropriate in all languages, but it might nonetheless be preferable to
introduce a new message for this purpose.
Comment 2 Ilmari Karonen 2006-05-28 22:17:58 UTC 
Created attachment 1854 [details]
Alternative patch, using a new "editold" message instead of "editsection"
Comment 3 JeLuF 2006-05-30 22:17:35 UTC 
fixed in revision 14486.
Note You need to log in before you can comment on or make changes to this bug.
First Last Prev Next    This bug is not in your last search results.

Navigation
Links