Page MenuHomePhabricator

Rollback links should not contain [ and ] characters
Closed, ResolvedPublic


The rollback link contains '[' and ']' characters. This prevents skin developers (e.g. Minerva) from skinning the link.

Screen Shot 2019-04-02 at 4.32.47 PM.png (78×544 px, 30 KB)

We've made good progress in core moving away from hardcoded parentheses and should do so here as well.

<span class="mw-rollback-link">[<a href="/w/index.php?title=Talk:Qatar&amp;action=rollback&amp;from=Jdlrobson&amp;token=ad3b6091b6a61e78107d8aa73f00557b5ca3ef2d%2B%5C" title="&quot;Rollback&quot; reverts the last contributor's edit(s) to this page in one click" data-mw="interface">rollback more than 10 edits</a>]</span>

Replication steps

You can enable rollback rights for all logged in users on your local instance using

$wgGroupPermissions['user']['rollback'] = true;

Event Timeline

Change 500864 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Do not hardcode square brackets in "rollback link"

I feel like there should be a parent task for getting rid of these “textual decorations” in general.

@Harej T205581 was the originating task, that grow in scope… ;)

Jdlrobson added a subscriber: kostajh.

@Volker_E or @kostajh any chance you can review the open patch? I just rebased.

Change 500864 merged by jenkins-bot:
[mediawiki/core@master] Do not hardcode square brackets in "rollback link"

Etonkovidova claimed this task.
Etonkovidova added a subscriber: Etonkovidova.

Checked in betalabs - square brackets are removed; rollback links work as expected.

Screen Shot 2019-07-22 at 1.30.59 PM.png (88×1 px, 52 KB)