Closed
Bug 1143725
Opened 10 years ago
Closed 10 years ago
First few paragraphs missing from parsed article
Categories
(Toolkit :: Reader Mode, defect, P3)
Toolkit
Reader Mode
Tracking
()
RESOLVED
FIXED
People
(Reporter: markh, Unassigned)
References
(Blocks 1 open bug, )
Details
+++ This bug was initially created as a clone of Bug #1125711 +++
I'm not sure if it is exactly the same as the above bug, so I created a new one.
In reader view, the first shown paragraph starts with "The Bill would require telecommunications service providers to store so-called “metadata” ..." - this is the 5th paragraph in the article.
Looking at the source, nothing stands out - the missing text is in a <div class="story-body ">, while the test being displayed is in a following <div id="read-more">
Comment 1•10 years ago
|
||
Glad you filed anyway, because this still reproduces after our changes to fix some of this. Want to look into this more later today.
Flags: needinfo?(gijskruitbosch+bugs)
Comment 2•10 years ago
|
||
This needs algorithm changes to not like the "read more" container so much and/or like another container even more. Leaving needinfo to look into this more.
Comment 3•10 years ago
|
||
Fixed in https://github.com/mozilla/readability/pull/50
Landed on fx-team as part of:
https://hg.mozilla.org/integration/fx-team/rev/8a292af9c827
(we've been marking things fixed based on the github status, so I'm closing this).
Status: NEW → RESOLVED
Closed: 10 years ago
Flags: needinfo?(gijskruitbosch+bugs)
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•