Change tags in some interfaces are missing parenthesis
Closed, ResolvedPublic
Actions

Description

Original title: Change tags in revision diffs are missing parenthesis

rMW1a5b26f0b604: Changelist "tags" item is wrapped in brackets via CSS

Screenshot from https://en.wikipedia.org/w/index.php?title=Rose_Nylund&curid=2935098&diff=875340192&oldid=868648183&diffmode=source&uselang=en

Seems that the stylesheet is not loaded on diffs.

QA steps

Make sure they show up on diffs
Make sure they show up on Special:Log
Should show on action=editchangetags, (needs permissions)
Should show on action=revisiondelete

Details

Subject	Repo	Branch	Lines +/-
Style change tags correctly on Special:Newpages	mediawiki/core	REL1_33	+2 -0
Style change tags correctly on Special:Newpages	mediawiki/core	master	+2 -0
Style change tags correctly on log extracts	mediawiki/core	master	+2 -0
Apply the pseudo-css selector rules to diff and changelist pages	mediawiki/core	master	+75 -41
Add mediawiki.interface.helpers.styles where it is needed	mediawiki/core	master	+10 -4
Tag markers should be wrapped in parentheses on diff page	mediawiki/core	master	+15 -0

Customize query in gerrit

Related Objects
Search...

Status	Subtype	Assigned	Task
Open		None	T158181 Aim for workflow equivalence for MediaWiki on desktop and mobile web
Duplicate		None	T265402 Diff links in mobile watchlist and recent changes go to page instead of diff
Resolved		Jdlrobson	T210168 [EPIC] Get rid of MobileFrontend's special pages for changes list where equivalents exist in core
Resolved	Goal	ovasileva	T198313 [GOAL] Advanced mobile contributions
Resolved		ovasileva	T216420 [EPIC] Display core version of history page
Resolved		None	T205581 History, Special:Contributions, Special:RecentChanges, Special:Watchlist should be fully skinnable - use pseudo elements for separating elements
Resolved		Jdlrobson	T212613 Change tags in some interfaces are missing parenthesis

Event Timeline

matej_suchanek created this task.Dec 25 2018, 7:23 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptDec 25 2018, 7:23 PM

Change 481935 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Tag markers should be wrapped in parentheses on diff page

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

gerritbot added a project: Patch-For-Review.Jan 2 2019, 7:39 PM

Jdlrobson added a project: User-Jdlrobson.Jan 2 2019, 7:39 PM

Jdlrobson added a parent task: T205581: History, Special:Contributions, Special:RecentChanges, Special:Watchlist should be fully skinnable - use pseudo elements for separating elements.

Jdlrobson added a subscriber: Volker_E.

Change 481935 merged by jenkins-bot:
[mediawiki/core@master] Tag markers should be wrapped in parentheses on diff page

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

Thanks @Volker_E for the merg! This is now fixed on master. On its way to production via the deployment schedule.

ReleaseTaggerBot added a project: MW-1.33-notes (1.33.0-wmf.12; 2019-01-08).Jan 2 2019, 9:00 PM

There's the same problem on Special:Log and also in other interfaces, like action=editchangetags, action=historysubmit or action=revisiondelete. I don't think this list is complete...

Hmm... how can I get a comprehensive list? I'll take a look at this tomorrow.

Thank you. Callers of [[ https://phabricator.wikimedia.org/source/mediawiki/browse/master/?grep=%3A%3AformatSummaryRow | ChangeTags::formatSummaryRow ]] should be a good start.

matej_suchanek renamed this task from Change tags in revision diffs are missing parenthesis to Change tags in some interfaces are missing parenthesis.Jan 3 2019, 8:52 PM

matej_suchanek removed a project: Patch-For-Review.

matej_suchanek updated the task description. (Show Details)

Sorry about this. If an urgent fix is needed I suggest using MediaWiki:Common.css in the meantime. I'm still trying to work out the best way to do this as there is no shared CSS module between these pages. It may make sense to generalise the CSS rule - either a css file loaded on all special pages or mediawiki.skinning.interface.

@Fomafix @Volker_E what if we put this in mediawiki.skinning.interface .. are brackets useful in non-special pages?

Change 483042 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Apply the mw-tag-markers rule to all special pages

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

gerritbot added a project: Patch-For-Review.Jan 9 2019, 12:41 AM

Change 483042 merged by jenkins-bot:
[mediawiki/core@master] Apply the pseudo-css selector rules to diff and changelist pages

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

ReleaseTaggerBot edited projects, added MW-1.33-notes (1.33.0-wmf.19; 2019-02-26); removed MW-1.33-notes (1.33.0-wmf.12; 2019-01-08).Feb 22 2019, 1:00 AM

On Thursday this should be fixed on all wikis.
Please test this out on mediawiki.org and let me know if this is broken anywhere!

Special:Log seems completely omitted from the fix.

Jdlrobson reopened this task as Open.Feb 27 2019, 8:10 PM

Jdlrobson removed a project: Patch-For-Review.

Jdlrobson updated the task description. (Show Details)

Looks like something went wrong in the rebase... my apologies for declaring this resolved prematurely.

Jdlrobson moved this task from Inbox to Next up on the User-Jdlrobson board.Feb 28 2019, 9:04 PM

Change 493513 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/core@master] Add mediawiki.interface.helpers.styles where it is needed

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

gerritbot added a project: Patch-For-Review.Feb 28 2019, 9:12 PM

Change 493513 merged by jenkins-bot:
[mediawiki/core@master] Add mediawiki.interface.helpers.styles where it is needed

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

This is now live on beta cluster. The code for using these tags is not very well organised so it's possible I've missed a few. Any testing experienced editors can provide with be most helpful, however I can confirm all the ones listed in task description are now working...

ReleaseTaggerBot edited projects, added MW-1.33-notes (1.33.0-wmf.21; 2019-03-12); removed MW-1.33-notes (1.33.0-wmf.19; 2019-02-26).Mar 5 2019, 2:04 PM

matej_suchanek closed this task as Resolved.Mar 19 2019, 4:31 PM

matej_suchanek removed a project: Patch-For-Review.

Change 500821 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/core@master] Style change tags correctly on log extracts

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

gerritbot added a project: Patch-For-Review.Apr 2 2019, 8:11 PM

Change 500821 merged by jenkins-bot:
[mediawiki/core@master] Style change tags correctly on log extracts

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

ReleaseTaggerBot edited projects, added MW-1.33-notes (1.33.0-wmf.25; 2019-04-09); removed MW-1.33-notes (1.33.0-wmf.21; 2019-03-12).Apr 2 2019, 9:01 PM

Change 517912 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/core@master] Style change tags correctly on Special:Newpages

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

Change 517913 had a related patch set uploaded (by Matěj Suchánek; owner: Matěj Suchánek):
[mediawiki/core@REL1_33] Style change tags correctly on Special:Newpages

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

Change 517912 merged by jenkins-bot:
[mediawiki/core@master] Style change tags correctly on Special:Newpages

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

Change 517913 merged by jenkins-bot:
[mediawiki/core@REL1_33] Style change tags correctly on Special:Newpages

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

ReleaseTaggerBot edited projects, added MW-1.33-notes, MW-1.34-notes (1.34.0-wmf.11; 2019-06-26); removed MW-1.33-notes (1.33.0-wmf.25; 2019-04-09).Jun 19 2019, 8:00 PM

Aklapper added a project: MediaWiki-Page-history.Sep 20 2019, 7:47 PM

Maintenance_bot removed a project: Patch-For-Review.Sep 20 2019, 8:11 PM

	F27724558: obrazek.png
	Dec 25 2018, 7:23 PM

Change tags in some interfaces are missing parenthesisClosed, ResolvedPublicActions