diff options
author | umbrae <umbrae@d4e419ec-0920-11de-bbfd-a7c1bc4c261e> | 2009-03-09 15:47:58 +0000 |
---|---|---|
committer | umbrae <umbrae@d4e419ec-0920-11de-bbfd-a7c1bc4c261e> | 2009-03-09 15:47:58 +0000 |
commit | d0743842804a6a8b853d45d81082f3c50f4d8fef (patch) | |
tree | 805b9f98ebd100cf22af6d36176eae1fd4583710 /js | |
parent | e1b7d7c642cc583a47af8738f2a14f87354e911b (diff) | |
download | readability-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-x | js/readability-start.js | 2 | ||||
-rwxr-xr-x | js/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
|