Open
Bug 1147626
Opened 10 years ago
Updated 2 years ago
[meta] Refine isProbablyReaderable algorithm
Categories
(Toolkit :: Reader Mode, defect, P3)
Toolkit
Reader Mode
Tracking
()
NEW
People
(Reporter: Margaret, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Keywords: meta, Whiteboard: [reader-mode-readability-algorithm])
In bug 1143844, we implemented an algorithm to only show the reader view button if a page has at least 5 <p> elements with a decent amount of text. It turns out that this might be a bit too aggressive, we should try to refine this.
Some testcases documented in https://github.com/mozilla/readability/issues/73:
1/ Wordpress:
Reader mode not activating:
https://blog.mozilla.org/blog/2015/03/16/please-welcome-david-bryant-vice-president-of-platform-engineering/
Reader mode activating:
https://blog.mozilla.org/blog/2015/03/03/unity-5-ships-and-brings-one-click-webgl-export-to-legions-of-game-developers/
2/ Dotclear
Reader mode not activating:
http://blog.mozfr.org/post/2015/03/La-lune-eclipse-le-soleil
Reader mode activating:
http://blog.mozfr.org/post/2015/03/Il-y-a-26-ans-naissait-le-Web-Coup-d-oeil-dans-le-retro
Comment 1•10 years ago
|
||
Setting qe-verify- for this bug, as the testing required here will be covered by the verification conducted for Bug 1139165.
Flags: qe-verify-
Updated•10 years ago
|
Priority: -- → P3
Reporter | ||
Comment 2•10 years ago
|
||
markh has a patch over in bug 1149859. Let's morph this into a meta bug, since there may be other things we want to try to make this logic better.
Updated•8 years ago
|
Whiteboard: [reader-mode-readability-algorithm]
Comment 3•8 years ago
|
||
Just wanted to comment on this. The reader button seems to sometimes appear but only display part of the page when reader mode is activated. For example, the article found here: http://lolajournal.com/7/lola_montes.html appears to be perfect for reader mode. However, when it's activated, only three paragraphs of the article are displayed, the ones starting with "Although Lola's..." and ending with "capitalist industry." These paragraphs are in the middle of the page, not the first few or last few paragraphs. This behavior is definitely a bug.
Updated•8 years ago
|
Summary: Refine isProbablyReaderable algorithm → [meta] Refine isProbablyReaderable algorithm
Comment 4•8 years ago
|
||
(In reply to lord.of.the.flies.0 from comment #3)
> Just wanted to comment on this. The reader button seems to sometimes appear
> but only display part of the page when reader mode is activated. For
> example, the article found here: http://lolajournal.com/7/lola_montes.html
> appears to be perfect for reader mode. However, when it's activated, only
> three paragraphs of the article are displayed, the ones starting with
> "Although Lola's..." and ending with "capitalist industry." These paragraphs
> are in the middle of the page, not the first few or last few paragraphs.
> This behavior is definitely a bug.
Please file a separate bug for this - this bug is not the right venue to investigate this further.
Comment 5•8 years ago
|
||
(In reply to :Gijs (gone until 3 jan) from comment #4)
> (In reply to lord.of.the.flies.0 from comment #3)
> > Just wanted to comment on this. The reader button seems to sometimes appear
> > but only display part of the page when reader mode is activated. For
> > example, the article found here: http://lolajournal.com/7/lola_montes.html
> > appears to be perfect for reader mode. However, when it's activated, only
> > three paragraphs of the article are displayed, the ones starting with
> > "Although Lola's..." and ending with "capitalist industry." These paragraphs
> > are in the middle of the page, not the first few or last few paragraphs.
> > This behavior is definitely a bug.
>
> Please file a separate bug for this - this bug is not the right venue to
> investigate this further.
Done, see 1325999. For some reason I do not see "Reader Mode" in the component field.
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•