Page MenuHomePhabricator

Unexpected section edit links appear on some pages with Parsoid
Closed, ResolvedPublic

Description

Unexpected section edit links appear on some pages with Parsoid, for example compare:

(There's also a difference in rendering with the heading underline, since the TemplateStyles designed for old parser output don't apply, that will probably need to be fixed on-wiki.)

Event Timeline

Change #1015436 had a related patch set uploaded (by Arlolra; author: Arlolra):

[mediawiki/services/parsoid@master] Literal html headings are never editable

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

ABreault-WMF triaged this task as Medium priority.
ABreault-WMF moved this task from Backlog to Code Review on the Content-Transform-Team-WIP board.

Change #1015436 merged by jenkins-bot:

[mediawiki/services/parsoid@master] Literal html headings don't get section edit links

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

Change #1017782 had a related patch set uploaded (by Isabelle Hurbain-Palatin; author: Isabelle Hurbain-Palatin):

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a25

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

Change #1017782 merged by jenkins-bot:

[mediawiki/vendor@master] Bump wikimedia/parsoid to 0.19.0-a25

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

Post deploy and purge, seems resolved.

(There's also a difference in rendering with the heading underline, since the TemplateStyles designed for old parser output don't apply, that will probably need to be fixed on-wiki.)

I'm not sure if anything was done on-wiki but the underline is no longer there. Could have been due to the wrong class being added before.

(There's also a difference in rendering with the heading underline, since the TemplateStyles designed for old parser output don't apply, that will probably need to be fixed on-wiki.)

I'm not sure if anything was done on-wiki but the underline is no longer there. Could have been due to the wrong class being added before.

Ah yes, previously the page had <div class="mw-heading"> wrappers on those headings, but they are not supposed to be there for tags written as <h2> in the wikitext (per T353489), and this has also been fixed by your change.