Last modified: 2014-09-01 19:19:40 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 T46242, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 44242 - "Undo" on "Difference between revisions" pages should have its own HTML class
"Undo" on "Difference between revisions" pages should have its own HTML class
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
History/Diffs (Other open bugs)
1.21.x
All All
: Low enhancement (vote)
: 1.24.0 release
Assigned To: danharibo+wt
: easy
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-01-22 15:25 UTC by Romaine
Modified: 2014-09-01 19:19 UTC (History)
5 users (show)

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


Attachments

Description Romaine 2013-01-22 15:25:51 UTC
A general wish, but especially for nl-Wikipedia: please give the undo button on diff pages a class to be able to hide it inside the personal preferences. On history pages this undo-button has its own class: class="mw-history-undo", we need one class for the diff pages as well.

We need this class soon.

Thanks!
Comment 1 Andre Klapper 2013-01-22 19:00:00 UTC
Thanks for reporting this. Consistency makes sense.

Clarifying terms for myself:
"history pages" = "Revision history"
"diff pages" = "Difference between revisions"

I can confirm this. Patches welcome. :)
Comment 2 Gerrit Notification Bot 2014-07-05 18:55:32 UTC
Change 144273 had a related patch set uploaded by Devanw:
add mw-diff-* classes to view diff page header links

https://gerrit.wikimedia.org/r/144273
Comment 3 Gerrit Notification Bot 2014-08-31 20:40:16 UTC
Change 144273 merged by jenkins-bot:
add mw-diff-* classes to view diff page header links

https://gerrit.wikimedia.org/r/144273
Comment 4 Jesús Martínez Novo (Ciencia Al Poder) 2014-09-01 19:19:40 UTC
Patch merged, so bug resolved :)

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


Navigation
Links