Maniphest T199855

WikidataPageBanner breaks Vector page subtitle
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Krinkle
	Jul 18 2018, 4:55 AM

Description

I've noticed this on test2.wikipedia.org where the extension was enabled.

The subtitle is meant to be inside a div#contentSub element, which is styled with a grey text color and a smaller font-size.

However, on wikis where WikidataPageBanner is enabled the subtitle is absent, and its contents are instead output inside a custom div.ext-wpb-pagebanner-subtitle element, which is missing the correct positioning and styling.

Traced it down to the following code:

Details

	Subject	Repo	Branch	Lines +/-
	Only move/restyle subtitle for pages where banner is present	mediawiki/extensions/WikidataPageBanner	wmf/1.32.0-wmf.19	+15 -6
	Only move/restyle subtitle for pages where banner is present	mediawiki/extensions/WikidataPageBanner	master	+15 -6

Customize query in gerrit

Related Objects

Mentioned Here: rEWDP7d39c374cc58: Allow skins to render subtitle after banner
T113642: Should be possible to print breadcrumbs above the heading to avoid reflows on Wikivoyage

Event Timeline

Krinkle created this task.Jul 18 2018, 4:55 AM

Restricted Application added a project: Wikidata. · View Herald TranscriptJul 18 2018, 4:55 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Krinkle added projects: Regression, Web-Team-Backlog.Jul 18 2018, 4:55 AM

Krinkle updated the task description. (Show Details)Jul 18 2018, 5:06 AM

Jdlrobson moved this task from Incoming to Needs Prioritization on the Web-Team-Backlog board.Jul 23 2018, 2:26 AM

This feels like an incorrectly configured WikidataPageBanner setup. Will look at setup on test wiki shortly.

I see 2 options here

Only do the subtitle fix when a banner exists (e.g. add `&& banner``` to https://gerrit.wikimedia.org/g/mediawiki/extensions/WikidataPageBanner/+/7d39c374cc589d9ecd714976d5508ca5f6b133c9/includes/WikidataPageBanner.hooks.php#62)
We disable the extension on test2 wiki

Will happily merge a patch for 1. Right now I am the only active maintainer of the extension and not wanting to self merge I can't submit a patch for 1.

Change 452004 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/WikidataPageBanner@master] Only move/restyle subtitle for pages where banner is present

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

gerritbot added a project: Patch-For-Review.Aug 10 2018, 9:00 PM

^ takes care of it, but as I'm the only maintainer I'm not sure if it will ever get reviewed :)

Jdlrobson moved this task from Needs Prioritization to Tracking on the Web-Team-Backlog board.Aug 16 2018, 9:37 PM

Jdlrobson edited projects, added Web-Team-Backlog (Tracking); removed Web-Team-Backlog.

Jdlrobson moved this task from Untriaged to Move to Backlog on the Web-Team-Backlog (Tracking) board.Aug 27 2018, 6:13 PM

Change 452004 merged by jenkins-bot:
[mediawiki/extensions/WikidataPageBanner@master] Only move/restyle subtitle for pages where banner is present

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

ReleaseTaggerBot added a project: MW-1.32-notes (WMF-deploy-2018-09-04 (1.32.0-wmf.20)).Aug 31 2018, 9:00 PM

Change 456757 had a related patch set uploaded (by Krinkle; owner: Jdlrobson):
[mediawiki/extensions/WikidataPageBanner@wmf/1.32.0-wmf.19] Only move/restyle subtitle for pages where banner is present

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

Change 456757 merged by jenkins-bot:
[mediawiki/extensions/WikidataPageBanner@wmf/1.32.0-wmf.19] Only move/restyle subtitle for pages where banner is present

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

ReleaseTaggerBot edited projects, added MW-1.32-notes (WMF-deploy-2018-08-28 (1.32.0-wmf.19)); removed MW-1.32-notes (WMF-deploy-2018-09-04 (1.32.0-wmf.20)).Aug 31 2018, 11:00 PM

Mentioned in SAL (#wikimedia-operations) [2018-08-31T23:00:34Z] <krinkle@deploy1001> Synchronized php-1.32.0-wmf.19/extensions/WikidataPageBanner/: I9dc9a4c1fb62c4 - T199855 (duration: 00m 51s)

Backported and verified on mwdebug and in prod.

WikidataPageBanner breaks Vector page subtitleClosed, ResolvedPublicActions

Description

Details

Related Objects

Event Timeline

WikidataPageBanner breaks Vector page subtitle
Closed, ResolvedPublic
Actions