Separate link class for anonymous users
Closed, ResolvedPublic

Description

Right now on Recent changes, links for logged-in users and IP's use the same "mw-userlink" class:

<a class="mw-userlink" title="User:USER123" href="/User:USER123">USER123</a>
<a class="mw-userlink" title="Special:Contributions/12.34.567.89" href="Special:Contributions/12.34.567.89">12.34.567.89</a>

I'm proposing that we give IP's a separate link class so that I can use CSS to make the IP's a different color to stand-out. This would make patrolling edits on Recent changes easier for me and my admins. Something like:

<a class="mw-userlink" title="User:USER123" href="/User:USER123">USER123</a>
<a class="mw-anonlink" title="Special:Contributions/12.34.567.89" href="Special:Contributions/12.34.567.89">12.34.567.89</a>


Version: 1.23.0
Severity: enhancement
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=40072
https://bugzilla.wikimedia.org/show_bug.cgi?id=43362

bzimport set Reference to bz43179.
Porplemontage created this task.Via LegacyDec 16 2012, 10:28 PM
Dereckson added a comment.Via ConduitDec 23 2012, 6:07 PM

[ Taking this bug. ]

To maximize coherence, this enhancement should be implemented to the following special pages:

  • [[Special:Contributions]]
  • [[Special:DeletedContributions]]
  • [[Special:RecentChanges]]
  • [[Special:Watchlist]]

It's also to be implemented on pages history.

To avoid to break any stuff relying on current CSS class, the CSS class for anonymous users should be added and not replaced:

<a class="mw-userlink" title="User:USER123" href="/User:USER123">USER123</a>
<a class="mw-userlink mw-anonuserlink" title="Special:Contributions/12.34.567.89"
href="Special:Contributions/12.34.567.89">12.34.567.89</a>

Would that be acceptable?

Porplemontage added a comment.Via ConduitDec 23 2012, 6:16 PM

Yes, that makes sense.

Aklapper added a comment.Via ConduitMar 20 2013, 3:45 PM

(In reply to comment #1 by Dereckson)

[ Taking this bug. ]

Dereckson: Do you still plan to work on this?

Note: As this has the 1.21.0 target milestone set, this needs to receive a patch. If this does not receive a patch in the next weeks, the Target Milestone will likely get removed.

Aklapper added a comment.Via ConduitApr 9 2013, 8:02 PM

Dereckson: Do you still plan to work on this?

Technical13 added a comment.Via ConduitFeb 23 2014, 11:09 PM

Is anyone still working on this? This would make it much easier than the current methods I've been using if($('.mw-userlink').search(/[12]?[\d]{1,2}\.[12]?[\d]{1,2}\.[12]?[\d]{1,2}\.[12]?[\d]{1,2}(\/[1-3]?[\d])?/)){ isIP=true; } which of course only catches IPv4 addresses.

gerritbot added a comment.Via ConduitFeb 24 2014, 12:25 AM

Change 115095 had a related patch set uploaded by Legoktm:
Add 'mw-anonuserlink' class for anonymous users

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

gerritbot added a comment.Via ConduitApr 17 2014, 8:21 PM

Change 115095 merged by jenkins-bot:
Add 'mw-anonuserlink' class for anonymous users

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

Add Comment

Column Prototype
This is a very early prototype of a persistent column. It is not expected to work yet, and leaving it open will activate other new features which will break things. Press "\" (backslash) on your keyboard to close it now.