Open Bug 1709154 Opened 4 years ago Updated 1 year ago

Reader Mode TTS treats entire article as a single paragraph due to useless div wrapper

Categories

(Toolkit :: Reader Mode, defect, P3)

Firefox 88
Desktop
All
defect

Tracking

()

People

(Reporter: morten, Unassigned)

Details

(Whiteboard: [reader-mode-narrate] [reader-mode-readability-algorithm] )

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/90.0.4430.93 Safari/537.36

Steps to reproduce:

Go to the following link and activate Reader Mode:
https://www.thenation.com/article/society/gender-silicon-valley/
Activate text-to-speech and try using the navigation buttons to navigate backwards and forward through the paragraphs.

I suspect this has something to do with the markup of the site not being parsed in a way that makes sense to the reader. Probably a good example of non-standard edge/stress case for the feature.

Actual results:

Once inside the article, the entire article is treated as one paragraph. Skip forward once and you get to the end of the article.

Expected results:

Clicking the forward/backwards buttons should navigate the user to the next/previous paragraph.

The Bugbug bot thinks this bug should belong to the 'Toolkit::Reader Mode' component, and is moving the bug to that component. Please revert this change in case you think the bot is wrong.

Component: Untriaged → Reader Mode
Product: Firefox → Toolkit

Works in private browsing mode. Seems like some JS wraps some stuff on the site and it ends up in a container, and the narrate stuff is pretty naive in that it expects the direct children of the article to be the individual units. Removing that assumption won't necessarily work, because e.g. paragraphs can also be nested, so it's just not that easy to determine what the "right" units are to chunk the article.

The easiest fix for this case is probably to get readability to strip out the extra div container.

Severity: -- → S3
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Unspecified → All
Priority: -- → P3
Hardware: Unspecified → Desktop
Summary: Reader Mode TTS treats entire article as a single paragraph → Reader Mode TTS treats entire article as a single paragraph due to useless div wrapper
Whiteboard: [reader-mode-narrate] [reader-mode-readability-algorithm]
You need to log in before you can comment on or make changes to this bug.