Articles on CNN frequently have an opening paragraph that is styled differently than others. Reader mode does not include this lead paragraph in its view. See, for example, http://money.cnn.com/2016/02/01/news/economy/poverty-inequality-united-states/index.html
Thanks for the report. This sounds like an issue with the Readability library, so I filed an issue here: https://github.com/mozilla/readability/issues/281
Cannot reproduce anymore. The webpage seems already changed. There is no opening paragraph there and the reader mode result is good. : http://money.cnn.com/2016/02/01/news/economy/poverty-inequality-united-states/index.html
(In reply to Evan Tseng [:evanxd][:愛聞插低] from comment #2) > Cannot reproduce anymore. The webpage seems already changed. There is no > opening paragraph there I see: The U.S. has long been heralded as a land of opportunity -- a place where anyone can succeed regardless of the economic class they were born into. on the page in a different font, and that paragraph does not make it into the reader mode result. Are you seeing something else? > and the reader mode result is good. > > : > http://money.cnn.com/2016/02/01/news/economy/poverty-inequality-united- > states/index.html
Gijs I see the same. I do know of cases where large sites to serve different markup to different regions.
I can reproduce the issue mentioned on Comment 3. Somehow the `<h2>The U.S. has long been heralded as a land of opportunity -- a place where anyone can succeed regardless of the economic class they were born into.</h2>` node is just removed by some kind of reason. Good thing is the algorithm chooses correct `topCandidate` (`<div id="storytext">`). Continue investigate the issue...
Added tests to investigate the issue. : https://github.com/mozilla/readability/pull/347/commits/077bca8721975efa607839f8c2756d2eee323f29
Sent a PR with the solution. Let's discuss it there. : https://github.com/mozilla/readability/pull/347/commits/a0f94b1869b5188dfad66d1d5cc8b6270c5bc4f2
Updated the patch to use a new solution to fix the issue. : https://github.com/mozilla/readability/pull/347/commits/64e97fead34ed567025109c2b6df0ae2d8a40db4
Fixed all test failures. : https://github.com/mozilla/readability/pull/347/commits/73a020d56675ba649a272d95e025404a9d7936f5
Updated patch for review comments: https://github.com/mozilla/readability/pull/347/commits/e6ae86bd9c4dc9d14f87d16f99f4a57b319ce4bb
We'll land it in m-c in the MozReview patch. : https://reviewboard.mozilla.org/r/109976/diff/2#index_header
Landed in m-c: https://hg.mozilla.org/mozilla-central/rev/d1bef3268b21