Page MenuHomePhabricator

Talk page discussion parsing is broken on mobile views with Parsoid read views enabled
Open, Needs TriagePublicBUG REPORT

Assigned To
None
Authored By
taavi
Mar 3 2024, 12:40 PM
Referenced Files
F42370943: image.png
Mar 3 2024, 7:35 PM
F42370938: image.png
Mar 3 2024, 7:35 PM
F42370921: image.png
Mar 3 2024, 7:35 PM
F42370924: image.png
Mar 3 2024, 7:35 PM
F42365850: image.png
Mar 3 2024, 12:40 PM
F42365839: image.png
Mar 3 2024, 12:40 PM
F42365834: image.png
Mar 3 2024, 12:40 PM

Description

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

  • Enable parsoid read views everywhere (or be on Wikitech)
  • Navigate to a talk page in mobile view

What happens?:
No discussions are shown, unless you click "Learn more about this page"

image.png (475×935 px, 43 KB)

image.png (763×928 px, 120 KB)

What should have happened instead?:
Discussions are shown normally

image.png (565×930 px, 71 KB)

Software version (skip for WMF-hosted wikis like Wikipedia):

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

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

This doesn't seem specific to DiscussionTools. I see several problems on mobile when using Parsoid read views.

Other problems include:

  • Not moving infobox after lead paragraph
  • Section collapsing does not work (this probably causes the DiscussionTools issue)

Also, I don't see the links to switch between Parsoid and legacy parser anywhere.

Yes, this is all blocked on T269499: [Epic] Make MobileFrontend compatible with Parsoid HTML -- I think focus should be on that task and this should either be closed as dupes of those (or made subtasks of that). Something the Web Team can figure out based on their triaging.

There is also T310544: Ensure MobileFrontend works with Parsoid read views for discussion tools in case there is any work to be done that is specific to DiscussionTools beyond what is done in T269499.

I think T262093 would help with the infobox issue.