mayoff / arc90labs-readability

Automatically exported from code.google.com/p/arc90labs-readability
0 stars 0 forks source link

Disabling/limiting multi-page mode #652

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I've discovered that Readability works nicely with http://bookworm.oreilly.com, 
an online Epub reader.

I am in favor of the new multipage feature for stitching together e.g. news 
articles, but in the context of reading novels, I would prefer to view one 
chapter a time. Using readability when viewing an ebook at Bookworm now loads 
the entire novel! It is entirely predictable and reasonable, but it makes me 
wish there was a configuration option to opt out of stitching (or to set the 
maxPages limit yourself).

Original issue reported on code.google.com by Jim.DeV...@gmail.com on 16 Sep 2010 at 6:16

GoogleCodeExporter commented 9 years ago
FYI, by adding the following bit around line 128 of readability.js, the 
bookmarklet can be customized with a readMaxPages variable to control the 
maximum number of pages to stitch together (set to 1 to prevent stitching - 
useful in the case I describe above).

if(typeof(readMaxPages) == 'number' && readMaxPages >= 1) {
    readability.maxPages = readMaxPages;
}

Original comment by Jim.DeV...@gmail.com on 18 Sep 2010 at 12:45

GoogleCodeExporter commented 9 years ago
Hi Jim,

Thanks for this. We'll give it some serious thought - we try to eschew too many 
options, but this might be one we need to provide.

-Chris Dary

Original comment by umb...@gmail.com on 18 Sep 2010 at 1:39