Probably due to invisible templates taking on a rendering, e.g.:
Description
Details
Related Objects
- Mentioned In
- T255131: Lede section edit broken on some pages
Event Timeline
Change 542246 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/MobileFrontend@master] identifyLeadParagraph: Ignore invisible nodes, slugs and whitespace
Change 542246 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] identifyLeadParagraph: Ignore invisible nodes, slugs and whitespace
@matmarex: This is not yet on en.wiki right? Why did @ReleaseTaggerBot not tag this? 😕
It's not on en.wiki yet, it should be included in the next week's train deployment. No idea about ReleaseTaggerBot.
Okay, checked it for the listed articles on en.wiki, scroll is now moving past the infobox for those articles.
It looks like in VisualEditorOverlay#scrollToLeadParagraph we have a section of (string)"0" but we are only checking for (number)0, so the identifyLeadParagraph code isn't running.
CC @matmarex who was working on the section code recently.
According to https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/VisualEditor/+/585029/ we should always treat sections as strings, so it looks like this case got missed.
Change 604689 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/MobileFrontend@master] Follow-up I8373a7a: Always treat sections as strings
Change 604689 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Follow-up I8373a7a: Always treat sections as strings