Fix IE8 <img> parser bug, add gitignore

armellarcier · Jun 30, 2011 · 64b67e3 · 64b67e3
1 parent 6ee1618
commit 64b67e3
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 0 deletions.
diff --git a/.gitignore b/.gitignore
@@ -0,0 +1 @@
+.tm_sync.config
diff --git a/dist/wysihtml5-0.1.0.js b/dist/wysihtml5-0.1.0.js
@@ -5500,6 +5500,17 @@ wysihtml5.utils.sanitizeHTML = (function() {
         newNode.setAttribute(attributeName, attributes[attributeName]);
       } catch(e) {}
     }
+
+    // IE8 sometimes loses the width/height attributes when those are set before the "src"
+    // so we make sure to set them again
+    if (attributes.src) {
+      if (typeof(attributes.width) !== "undefined") {
+        newNode.setAttribute("width", attributes.width);
+      }
+      if (typeof(attributes.height) !== "undefined") {
+        newNode.setAttribute("height", attributes.height);
+      }
+    }
   }
 
   /**

diff --git a/src/utils/sanitize_html.js b/src/utils/sanitize_html.js
@@ -297,6 +297,17 @@ wysihtml5.utils.sanitizeHTML = (function() {
         newNode.setAttribute(attributeName, attributes[attributeName]);
       } catch(e) {}
     }
+
+    // IE8 sometimes loses the width/height attributes when those are set before the "src"
+    // so we make sure to set them again
+    if (attributes.src) {
+      if (typeof(attributes.width) !== "undefined") {
+        newNode.setAttribute("width", attributes.width);
+      }
+      if (typeof(attributes.height) !== "undefined") {
+        newNode.setAttribute("height", attributes.height);
+      }
+    }
   }
 
   /**