Page MenuHomePhabricator

Add class mw-parser-output to Parsoid's output
Closed, ResolvedPublic

Description

This was changed in T37247: content-holding <div> should only contain the page text for the MW parser but no task was created for the other parser, whoops.

Related Objects

StatusAssignedTask
OpenNone
OpenNone
ResolvedNone
DuplicateNone
OpenNone
OpenNone
OpenNone
DuplicateNone
OpenNone
OpenNone
Resolved Jdlrobson
DuplicateNone
DuplicateNone
OpenNone
OpenNone
StalledNone
InvalidNone
OpenNone
OpenNone
ResolvedTheDJ
ResolvedTheDJ
InvalidNone
OpenNone
ResolvedTheDJ
OpenNone
Resolved Jdlrobson
Open Jdlrobson
Open Jdlrobson
ResolvedTgr
ResolvedEsanders
ResolvedEsanders
Resolvedssastry

Event Timeline

ssastry added a subscriber: ssastry.May 8 2017, 8:38 PM

Tim mentioned this last Thu when we were syncing up .. I was planning to take a look soon, but thanks for the ticket!

Anomie added a comment.May 8 2017, 8:39 PM

This was changed in T37247: content-holding <div> should only contain the page text for the MW parser but no task was created for the other parser, whoops.

To be fair, I did ask (in T37247#3181097 and on wikitech-l) for people who know about how the Parsoid ecosystem works to say whether that would be necessary or desirable, but no one replied.

This was changed in T37247: content-holding <div> should only contain the page text for the MW parser but no task was created for the other parser, whoops.

To be fair, I did ask (in T37247#3181097 and on wikitech-l) for people who know about how the Parsoid ecosystem works to say whether that would be necessary or desirable, but no one replied.

It was on my list for a face-to-face discussion in Vienna as to whether this was a good idea…

Change 352711 had a related patch set uploaded (by Subramanya Sastry; owner: Subramanya Sastry):
[mediawiki/services/parsoid@master] Add mw-parser-output class to Parsoid's output

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

ssastry moved this task from Backlog to In Progress on the Parsoid board.May 8 2017, 10:46 PM
ssastry triaged this task as Normal priority.
ssastry claimed this task.

Change 352711 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Add mw-parser-output class to Parsoid's output

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

ssastry closed this task as Resolved.May 10 2017, 3:25 PM