Section collapsing applies to namespaces it should not apply to in mobile
Closed, ResolvedPublic3 Estimated Story PointsBUG REPORT
Actions

Description

Steps to replicate the issue (include links if applicable):

What happens?:
In Parsoid the summary section is collapsed. In legacy parser it isn't.

What should have happened instead?:
It was a previous product decision to not apply section collapsing to certain pages frequented by editors.

This was achieved using the MFNamespacesWithoutCollapsibleSections configuration option. While it appears to apply to legacy parser it also needs to be used in MobileFrontend's onArticleParserOptions hook.

Software version (on Special:Version page; skip for WMF-hosted wikis like Wikipedia):

Other information (browser name/version, screenshots, etc.):

Details

Related Changes in Gerrit:

	Subject	Repo	Branch	Lines +/-
	Add namespace-specific collapsible section handlng for Parsoid mobile	mediawiki/extensions/MobileFrontend	wmf/1.46.0-wmf.11	+57 -11
	Add namespace-specific collapsible section handlng for Parsoid mobile	mediawiki/extensions/MobileFrontend	master	+57 -11

Customize query in gerrit

Related Objects

Mentioned In: T415451: MobileFrontend tests that depend on ext:ParserMigration are currently failing (and also don't seem to be being run in CI)
T413404: [Minerva TOC] All sections are always expanded
T374883: Section collapsing should not apply to certain pages
Mentioned Here: T374883: Section collapsing should not apply to certain pages

Event Timeline

Jdlrobson-WMF created this task.Oct 20 2025, 11:31 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 20 2025, 11:31 PM

Jdlrobson-WMF moved this task from Untriaged to MobileFormatter on the MobileFrontend board.Oct 21 2025, 3:42 PM

Jdlrobson-WMF edited projects, added MobileFrontend (MobileFormatter); removed MobileFrontend.

ovasileva triaged this task as Medium priority.Oct 22 2025, 11:46 AM

ovasileva moved this task from Incoming/Inbox to Backlog on the Reader Growth Team board.

cscott moved this task from Backlog to Tracking on the Content-Transform-Team board.Oct 23 2025, 2:50 PM

Jdlrobson-WMF renamed this task from Section collapsing applies to pages it should not apply to in mobile to Section collapsing applies to namespaces it should not apply to in mobile.Oct 23 2025, 4:55 PM

Jdlrobson-WMF moved this task from Backlog to Blockers on the Readers Essential Work 2025 (WE6.3.4 Fix bugs within the reading experience across platforms for Parsoid Read Views) board.

Jdlrobson-WMF updated the task description. (Show Details)

Jdlrobson-WMF mentioned this in T374883: Section collapsing should not apply to certain pages.

egardner moved this task from Backlog to Needs Refinement on the Reader Growth Team board.Dec 10 2025, 5:23 PM

HSwan-WMF set the point value for this task to 3.Dec 10 2025, 5:32 PM

HSwan-WMF moved this task from Needs Refinement to Ready on the Reader Growth Team board.Dec 10 2025, 5:50 PM

lwatson claimed this task.Dec 15 2025, 5:21 PM

lwatson moved this task from Ready to Sprint 6 (Dec 10 - EOY) Q2 25/26 on the Reader Growth Team board.Dec 15 2025, 5:24 PM

lwatson edited projects, added Reader Growth Team (Sprint 6 (Dec 10 - EOY) Q2 25/26); removed Reader Growth Team.

lwatson moved this task from Committed to Doing on the Reader Growth Team (Sprint 6 (Dec 10 - EOY) Q2 25/26) board.Dec 17 2025, 6:49 PM

Change #1219599 had a related patch set uploaded (by LWatson; author: LWatson):

[mediawiki/extensions/MobileFrontend@master] Disable collapsible sections in certain namespaces for mobile and Parsoid

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

gerritbot added a project: Patch-For-Review.Dec 18 2025, 3:37 PM

There was a hiccup with CI checks. Phan test failed because of an undeclared method in SpecialMobileEditWatchlist (SpecialMobileEditWatchlist::getWatchlistInfo). This patch restored the method and now CI checks pass.

Hm; the patch logic does prevent a call to $parserOptions->setCollapsibleSections() but this doesn't seem to actually prevent section collapsing on parsoid renderings.

I still get collapsible sections on file description pages when testing this patch. Steps to reproduce:

ensure that MobileFrontend and ParserMigration are installed locally
download a local copy of File:Münster, Schlossplatz, Herbstsend, Kettenkarussell -- 2024 -- 6459.jpg from Commons
upload it to your local test wiki
edit the File: page to include at least one section, for instance:

xxx

== Summary ==
adsfdfsadfsadsfafdsafdsadsffaaaafdsafdsfds

view the page in mobile view, both without and with useparsoid=1

Expected results:

both legacy and parsoid renderings should not provide for collapsible sections

Actual results:

legacy rendering has no collapsible sections but parsoid has collapsible "Summary" section both with and without this patch

It's possible I'm misunderstanding the purpose of this patch?

SherryYang-WMF moved this task from Sprint 6 (Dec 10 - EOY) Q2 25/26 to Sprint 1 (Jan 6 - Feb 3) Q3 25/26 on the Reader Growth Team board.Jan 6 2026, 4:50 PM

SherryYang-WMF edited projects, added Reader Growth Team (Sprint 1 (Jan 6 - Feb 3) Q3 25/26); removed Reader Growth Team (Sprint 6 (Dec 10 - EOY) Q2 25/26).

egardner moved this task from Committed to Code Review on the Reader Growth Team (Sprint 1 (Jan 6 - Feb 3) Q3 25/26) board.Jan 6 2026, 8:01 PM

lwatson moved this task from Code Review to Doing on the Reader Growth Team (Sprint 1 (Jan 6 - Feb 3) Q3 25/26) board.Jan 9 2026, 2:53 PM

lwatson mentioned this in T413404: [Minerva TOC] All sections are always expanded.Jan 13 2026, 7:05 PM

lwatson moved this task from Doing to Code Review on the Reader Growth Team (Sprint 1 (Jan 6 - Feb 3) Q3 25/26) board.Jan 14 2026, 1:12 PM

Change #1219599 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Add namespace-specific collapsible section handlng for Parsoid mobile

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

Jdlrobson-WMF moved this task from Code Review to QA on the Reader Growth Team (Sprint 1 (Jan 6 - Feb 3) Q3 25/26) board.Jan 14 2026, 5:16 PM

Maintenance_bot removed a project: Patch-For-Review.Jan 14 2026, 5:31 PM

Change #1227849 had a related patch set uploaded (by Jdlrobson; author: LWatson):

[mediawiki/extensions/MobileFrontend@wmf/1.46.0-wmf.11] Add namespace-specific collapsible section handlng for Parsoid mobile

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

gerritbot added a project: Patch-For-Review.Jan 16 2026, 4:48 PM

In T407815#11493513, @bvibber wrote:
Hm; the patch logic does prevent a call to $parserOptions->setCollapsibleSections() but this doesn't seem to actually prevent section collapsing on parsoid renderings.

I still get collapsible sections on file description pages when testing this patch. Steps to reproduce:

ensure that MobileFrontend and ParserMigration are installed locally

download a local copy of File:Münster, Schlossplatz, Herbstsend, Kettenkarussell -- 2024 -- 6459.jpg from Commons

upload it to your local test wiki

edit the File: page to include at least one section, for instance:
xxx

== Summary ==
adsfdfsadfsadsfafdsafdsadsffaaaafdsafdsfds
view the page in mobile view, both without and with useparsoid=1

Expected results:

both legacy and parsoid renderings should not provide for collapsible sections

Actual results:

legacy rendering has no collapsible sections but parsoid has collapsible "Summary" section both with and without this patch

I checked the File:Münster, Schlossplatz, Herbstsend, Kettenkarussell -- 2024 -- 6459.jpg on enwiki beta - did not see any issues - with &useparsoid=1 the sections are not collapsed.

It seems that in current production enwiki wmf.11, File pages that have the template Non-free_use_rationale_video_cover (hastemplate:"Non-free_use_rationale_video_cover" - enwiki betalabs doesn't have File pages with that template) display the issue. It''s worth to check several examples after deployment the fix to production.

Note:
In Desktop mode there is an interesting option in the Tools menu - "Expand all" -

Screenshot 2026-01-16 at 5.36.29 PM.png (1×2 px, 2 MB)

As far as I could see, the "Expand all" option doesn't do anything and it's not present in mobile view.

A_smart_kitten mentioned this in T415451: MobileFrontend tests that depend on ext:ParserMigration are currently failing (and also don't seem to be being run in CI).Sat, Jan 24, 9:11 PM

matthiasmullie closed this task as Resolved.Mon, Jan 26, 5:04 PM

Change #1227849 abandoned by Jdlrobson: