New issue
Advanced search Search tips

Issue 678656 link

Starred by 0 users
Status: Assigned
Owner:
wychen@chromium.org
Cc:
mdjones@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Make page mobile friendly ignores articles on daringfireball.net

Project Member Reported by amineer@chromium.org, Jan 5 2017 
Internal bug: b/33501308

To repro:
1) Visit daringfireball.net
2) Hit "Make page mobile-friendly"

Expected result:
See all recent articles

Actual result:
Some articles aren't shown

Repro: 2/2

Internal bug for M54, I can repro on current canary M57.

mdjones@, I'm not sure what component this should go into - can you help triage this?

Comment 1 by mdjones@chromium.org, Jan 5 2017

Cc: mdjones@chromium.org
Components: -UI>Browser>Mobile UI>Browser>ReaderMode
Owner: wychen@chromium.org
This is a weird case because it's many articles on a single page instead of just one. Assigning to wychen as he is actively working on reader.

Comment 2 by wychen@chromium.org, Jan 6 2017 

Thanks for reporting a bug!

The main page of daringfireball.net should not be considered distillable because it doesn't contain exactly one article. This looks more like a bug of distillability model.

However, it would be tricky to fix the distillability model because the "articles" don't have the usual clue in the DOM structure, like being wrapped in <article> or other annotations.

For URLs with individual articles, like http://daringfireball.net/2016/11/new_touch_bar_equipped_macbook_pros, it works as expected.
►

Sign in to add a comment