Page MenuHomePhabricator

Section collapsing in Parsoid version not resilient in case of slow connections
Open, MediumPublic3 Estimated Story PointsBUG REPORT

Description

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

What happens?:
It doesn't open. The page is unreadable

What should have happened instead?:
The sections should open.

The old legacy parser achieves this by a function MakeSectionsTransform::interimTogglingSupport.
For Parsoid add something like the following inline script:

window.mfTempClickHandler = document.addEventListener('click', ( ev ) => {
	const heading = ev.target.closest( '.mw-heading');
    heading.nextSibling.classList.add('mf-collapsible-content')
});

... and remove it inside src/mobile.init/toggling.js like so:

			document.removeEventListener( 'click', window.mfTempClickHandler );

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

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

Event Timeline

egardner lowered the priority of this task from High to Medium.Wed, Feb 4, 6:16 PM
egardner set the point value for this task to 3.

Change #1240789 had a related patch set uploaded (by Bvibber; author: Bvibber):

[mediawiki/extensions/MobileFrontend@master] WIP fix for temp section open during slow loads on Parsoid

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

Change #1240789 merged by jenkins-bot:

[mediawiki/extensions/MobileFrontend@master] Fix for temp section open during slow loads on Parsoid

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

Checked on enwiki beta - the issue is fixed; howver it'd be better to check on prod, wmf.18since the prod env and articles are different.

Below is the gif from emwiki wmf.17 - the fix is NOT there yet, The gif illustrates the issue:

In the gif: throttling 3G, usepaarsoid=1 - https://en.wikipedia.org/wiki/Alps/usepaarsoid=1

collapsing section_parsoid1 4.gif (693×372 px, 32 KB)

Good articles to check