Fix issues in IE8+9 (fixes #59)

tiff · Sep 24, 2012 · f75cb5a · f75cb5a · tiff · Sep 24, 2012
1 parent cab478a
commit f75cb5a
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 1 deletion.
diff --git a/src/browser.js b/src/browser.js
@@ -337,6 +337,13 @@ wysihtml5.browser = (function() {
      */
     hasInsertNodeIssue: function() {
       return isOpera;
+    },
+
+    /**
+     * IE 8+9 don't fire the focus event of the <body> when the iframe gets focused (even though the caret gets set into the <body>)
+     */
+    hasIframeFocusIssue: function() {
+      return isIE;
     }
   };
 })();
diff --git a/src/views/composer.observe.js b/src/views/composer.observe.js
@@ -146,7 +146,24 @@
         event.preventDefault();
       }
     });
+
+    // --------- IE 8+9 focus the editor when the iframe is clicked (without actually firing the 'focus' event on the <body>) ---------
+    if (browser.hasIframeFocusIssue()) {
+      dom.observe(this.iframe, "focus", function() {
+        setTimeout(function() {
+          if (that.doc.querySelector(":focus") !== that.element) {
+            that.focus();
+          }
+        }, 0);
+      });
 
+      dom.observe(this.element, "blur", function() {
+        setTimeout(function() {
+          that.selection.getSelection().removeAllRanges();
+        }, 0);
+      });
+    }
+
     // --------- Show url in tooltip when hovering links or images ---------
     var titlePrefixes = {
       IMG: "Image: ",

diff --git a/src/views/composer.style.js b/src/views/composer.style.js
@@ -47,7 +47,7 @@
       ],
       ADDITIONAL_CSS_RULES = [
         "html                 { height: 100%; }",
-        "body                 { min-height: 100%; padding: 0; margin: 0; margin-top: -1px; padding-top: 1px; }",
+        "body                 { height: 100%; padding: 1px 0 0 0; margin: -1px 0 0 0; }",
         "body > p:first-child { margin-top: 0; }",
         "._wysihtml5-temp     { display: none; }",
         wysihtml5.browser.isGecko ?