summaryrefslogtreecommitdiff
path: root/js
diff options
context:
space:
mode:
authorumbrae <umbrae@d4e419ec-0920-11de-bbfd-a7c1bc4c261e>2009-03-09 15:47:58 +0000
committerumbrae <umbrae@d4e419ec-0920-11de-bbfd-a7c1bc4c261e>2009-03-09 15:47:58 +0000
commitd0743842804a6a8b853d45d81082f3c50f4d8fef (patch)
tree805b9f98ebd100cf22af6d36176eae1fd4583710 /js
parente1b7d7c642cc583a47af8738f2a14f87354e911b (diff)
downloadreadability-simple-d0743842804a6a8b853d45d81082f3c50f4d8fef.tar.bz2
readability-simple-d0743842804a6a8b853d45d81082f3c50f4d8fef.zip
Removing debug messages, formatting comments to standard, moving to readability.js
git-svn-id: http://arc90labs-readability.googlecode.com/svn/trunk@17 d4e419ec-0920-11de-bbfd-a7c1bc4c261e
Diffstat (limited to 'js')
-rwxr-xr-xjs/readability-start.js2
-rwxr-xr-xjs/readability.js (renamed from js/readability-0.1.js)42
2 files changed, 12 insertions, 32 deletions
diff --git a/js/readability-start.js b/js/readability-start.js
index 834984e..ae90443 100755
--- a/js/readability-start.js
+++ b/js/readability-start.js
@@ -6,7 +6,7 @@ var baseHref = window.location.toString().match(/.*\//);
var linkStringStart = "javascript:(function(){";
-var linkStringEnd = "';_readability_script=document.createElement('SCRIPT');_readability_script.type='text/javascript';_readability_script.src='" + baseHref + "js/readability-0.1.js?x='+(Math.random());document.getElementsByTagName('head')[0].appendChild(_readability_script);_readability_css=document.createElement('LINK');_readability_css.rel='stylesheet';_readability_css.href='" + baseHref + "css/readability.css';_readability_css.type='text/css';_readability_css.media='screen';document.getElementsByTagName('head')[0].appendChild(_readability_css);_readability_print_css=document.createElement('LINK');_readability_print_css.rel='stylesheet';_readability_print_css.href='" + baseHref + "css/readability-print.css';_readability_print_css.media='print';_readability_print_css.type='text/css';document.getElementsByTagName('head')[0].appendChild(_readability_print_css);})();";
+var linkStringEnd = "';_readability_script=document.createElement('SCRIPT');_readability_script.type='text/javascript';_readability_script.src='" + baseHref + "js/readability.js?x='+(Math.random());document.getElementsByTagName('head')[0].appendChild(_readability_script);_readability_css=document.createElement('LINK');_readability_css.rel='stylesheet';_readability_css.href='" + baseHref + "css/readability.css';_readability_css.type='text/css';_readability_css.media='screen';document.getElementsByTagName('head')[0].appendChild(_readability_css);_readability_print_css=document.createElement('LINK');_readability_print_css.rel='stylesheet';_readability_print_css.href='" + baseHref + "css/readability-print.css';_readability_print_css.media='print';_readability_print_css.type='text/css';document.getElementsByTagName('head')[0].appendChild(_readability_print_css);})();";
$(document).ready(function() {
diff --git a/js/readability-0.1.js b/js/readability.js
index 7250433..1c92e3f 100755
--- a/js/readability-0.1.js
+++ b/js/readability.js
@@ -28,13 +28,6 @@
document.body.insertBefore(objOverlay, document.body.firstChild);
})()
-/* Remove this and any dbg calls before release to bring down file size. */
-function dbg(text)
-{
- if(typeof console != 'undefined')
- console.log(text);
-}
-
function grabArticle() {
var allParagraphs = document.getElementsByTagName("p");
var topDivCount = 0;
@@ -58,7 +51,7 @@ function grabArticle() {
for (var j=0; j < allParagraphs.length; j++) {
parentNode = allParagraphs[j].parentNode;
- /* Initialize readability data */
+ // Initialize readability data
if(typeof parentNode.readability == 'undefined')
{
parentNode.readability = {"contentScore": 0};
@@ -76,42 +69,29 @@ function grabArticle() {
parentNode.readability.contentScore += 25;
}
- /* Add a point for the paragraph found */
+ // Add a point for the paragraph found
if(getInnerText(allParagraphs[j]).length > 10)
parentNode.readability.contentScore++;
- /* Add points for any commas within this paragraph */
- dbg("Current paragraph has " + getCharCount(allParagraphs[j]) + " commas.");
+ // Add points for any commas within this paragraph
parentNode.readability.contentScore += getCharCount(allParagraphs[j]);
}
- /* Assignment from index for performance. See http://www.peachpit.com/articles/article.aspx?p=31567&seqNum=5 */
+ // Assignment from index for performance. See http://www.peachpit.com/articles/article.aspx?p=31567&seqNum=5
for(nodeIndex = 0; (node = document.getElementsByTagName('*')[nodeIndex]); nodeIndex++)
- {
- if(typeof node.readability != 'undefined')
- {
- dbg('Found a node with a content score of ' + node.readability.contentScore);
- if(topDiv == null || node.readability.contentScore > topDiv.readability.contentScore)
- {
- dbg('Found a more fit node. Setting topDiv.' + node.className);
- topDiv = node;
- }
- }
- }
+ if(typeof node.readability != 'undefined' && (topDiv == null || node.readability.contentScore > topDiv.readability.contentScore))
+ topDiv = node;
// REMOVES ALL STYLESHEETS ...
- for (var k=0;k < document.styleSheets.length; k++) {
- if (document.styleSheets[k].href != null && document.styleSheets[k].href.lastIndexOf("readability") == -1) {
+ for (var k=0;k < document.styleSheets.length; k++)
+ if (document.styleSheets[k].href != null && document.styleSheets[k].href.lastIndexOf("readability") == -1)
document.styleSheets[k].disabled = true;
- }
- }
+
// Remove all style tags in head (not doing this on IE) :
var styleTags = document.getElementsByTagName("style");
- for (var j=0;j < styleTags.length; j++) {
- if (navigator.appName != "Microsoft Internet Explorer") {
+ for (var j=0;j < styleTags.length; j++)
+ if (navigator.appName != "Microsoft Internet Explorer")
styleTags[j].textContent = "";
- }
- }
cleanStyles(topDiv); // Removes all style attributes
topDiv = killDivs(topDiv); // Goes in and removes DIV's that have more non <p> stuff than <p> stuff