Closed Bug 794480 Opened 12 years ago Closed 9 years ago

Reader Mode not displaying whole article

Categories

(Toolkit :: Reader Mode, defect, P3)

defect
Points:
5

Tracking

()

RESOLVED WORKSFORME
Tracking Status
firefox16 - wontfix
firefox17 - affected
firefox18 --- affected

People

(Reporter: krudnitski, Unassigned)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

On my Asus Transformer Prime using Fx 16 Beta, went to http://southernfood.about.com/od/crockpotporkandham/r/r114c3b.htm (don't judge the recipe selection - although can tell you how it turns out later).

Pressed the Reader icon and it displayed in Reader mode.

ER: the whole article gets displayed
AR: only a portion of the recipe got displayed (see attached screen shot - that's scrolled down to the very bottom of the article)
another (cooking / baking) page where this happens. webpage: http://m.epicurious.com/recipes/food/views/Maple-Pumpkin-Pie-104200. Screen shot of 'reader mode' result attached. And for anyone interested, this is a great recipe.
Is this a recent regression in the Reader Mode feature? If not, given where we are in the cycle, we won't be taking a fix for FF16. If this was a more prevalent issue in common use cases, we would have heard about it sooner.
I've seen this on many (if not most) Wikipedia articles also. For example, http://en.wikipedia.org/wiki/Mountain_View,_CA and http://en.wikipedia.org/wiki/Mozilla_corporation both exhibit this problem.

Due to the way these pages are broken up, our reader mode parser has trouble distinguishing which blocks in the page are part of the article. I don't believe this is a regression.
Wontfixing for FF16 because this missed our final beta.
No one has piped up to call this a regression and with no one assigned it doesn't appear to be a Reader Mode priority.  Untracking for 17 but if a low-risk fix becomes known we'll consider for uplift to branches depending on timing during the cycle.
Priority: -- → P1
Component: Reader View → Reader Mode
Product: Firefox for Android → Toolkit
Version: Firefox 16 → unspecified
Flags: qe-verify?
Flags: firefox-backlog+
Blocks: 1132074
(In reply to Karen Rudnitski [:kar] from comment #2)
> another (cooking / baking) page where this happens. webpage:
> http://m.epicurious.com/recipes/food/views/Maple-Pumpkin-Pie-104200. Screen
> shot of 'reader mode' result attached. And for anyone interested, this is a
> great recipe.

I don't even see the reader view button appear for this site, but article content for it does appear when I view this with readable-proxy, so it seems like something else is going wrong here.

However, when I load this in readable-proxy, I do see the ingredient list is missing, so we could focus on fixing that in this bug.

(In reply to Brian Nicholson (:bnicholson) from comment #4)
> I've seen this on many (if not most) Wikipedia articles also. For example,
> http://en.wikipedia.org/wiki/Mountain_View,_CA and
> http://en.wikipedia.org/wiki/Mozilla_corporation both exhibit this problem.

Both of these pages work for me.

Maybe something has changed since this bug was filed?
Points: --- → 5
Updating stale priorities to reflect priority in current reading list / reader view work.

(I see the issue reported in comment 0 -- the entire ingredient list is missing, and the reader view starts with "Preparation".)
OS: Android → All
Priority: P1 → P3
Hardware: Other → All
No longer blocks: 1132074
The reader view button no longer shows up with the recipes in comment 0 / comment 2.

I'm going to call this WFM (or arguably WONTFIX). I think recipe-like articles are going to be hard to support with reader view filter heuristics, because they're generally short bits of text (list of ingredients, a short paragraph or two of directions). That's hard to distinguish from a random web page with a few brief bits of text, which generally isn't useful to expose to reader view.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: