From 675f54b98b774c533be17f82b72a1a32028a3457 Mon Sep 17 00:00:00 2001
From: Nick White <git@njw.me.uk>
Date: Sun, 14 Aug 2011 20:31:59 +0100
Subject: Disable inline style tags

---
 TODO          | 12 ++++--------
 simplyread.js |  2 ++
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/TODO b/TODO
index 4ba3e07..b4f869b 100644
--- a/TODO
+++ b/TODO
@@ -1,12 +1,4 @@
 core:
-	inline <style> tags should be dropped. p.p1 for example is
-	specified in html here:
-	http://www.fringereview.co.uk/pageView.php?pagename=Latitude%20Festival%202011
-		note that as this needs to also drop its children, we can't
-		just use regex. whether using js remove element function is
-		safe (e.g. will not affect original html when toggled) is
-		unsure. doesn't look like there's a 'disable' attribute for
-		the tag, so it's our best hope.
 	consider counting <br />\s<br \> as a <p> as least some blogspot pages need it
 		http://davidz25.blogspot.com/2011/06/writing-c-library-part-1.html
 
@@ -76,4 +68,8 @@ gecko:
 chromium:
 	it would be nice to use the svg icon directly (smaller, faster, neater)
 		currently not supported (http://code.google.com/p/chromium/issues/detail?id=29683)
+		get the following error:
+		[18840:18840:7322270382:ERROR:extension_error_reporter.cc(55)]
+		Extension error:
+		Package is invalid. Details: 'Could not decode theme image.'
 	would be nice to have permission warning reflect the fact that no disk or network activity is done; that this is self-contained and lovely. unfortunately it isn't fine-grained enough, and we need all "tabs" permission to inject the script into any page
diff --git a/simplyread.js b/simplyread.js
index 5eb4614..dff7019 100755
--- a/simplyread.js
+++ b/simplyread.js
@@ -53,6 +53,8 @@ function simplyread()
 	fresh.innerHTML = fresh.innerHTML.replace(/<\/?font[^>]*>/g, "");
 	fresh.innerHTML = fresh.innerHTML.replace(/style="[^"]*"/g, "");
 	fresh.innerHTML = fresh.innerHTML.replace(/<\/?a[^>]*>/g, "");
+	fresh.innerHTML = fresh.innerHTML.replace(/<\/?span[^>]*>/g, "");
+	fresh.innerHTML = fresh.innerHTML.replace(/<style[^>]*>/g, "<style media=\"aural\">");
 	
 	for (var i = 0; i < doc.styleSheets.length; i++)
 		doc.styleSheets[i].disabled = true;
-- 
cgit v1.2.3