Fix #144 Does not work on IE8 - polyfills not initiated to iframe window

Fix Placeholder and InternetExplorer #55
Voog · Feb 17, 2015 · ad502cc · ad502cc
1 parent f04e11e
commit ad502cc
Show file tree

Hide file tree

Showing 6 changed files with 290 additions and 339 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -4,8 +4,8 @@ module.exports = function(grunt) {
 
   // List required source files that will be built into wysihtml5x.js
   var base = [
-    "src/polyfills.js",
     "src/wysihtml5.js",
+    "src/polyfills.js",
     "node_modules/rangy/lib/rangy-core.js",
     "node_modules/rangy/lib/rangy-selectionsaverestore.js",
     "lib/base/base.js",

diff --git a/src/dom/sandbox.js b/src/dom/sandbox.js
@@ -195,6 +195,10 @@
         this._unset(iframeDocument, "cookie", "", true);
       }
 
+      if (wysihtml5.polyfills) {
+        wysihtml5.polyfills(iframeWindow, iframeDocument);
+      }
+
       this.loaded = true;
 
       // Trigger the callback

diff --git a/src/dom/simulate_placeholder.js b/src/dom/simulate_placeholder.js
@@ -33,7 +33,7 @@
           dom.removeClass(view.element, CLASS_NAME);
         },
         set = function() {
-          if (view.isEmpty()) {
+          if (view.isEmpty() && !view.placeholderSet) {
             view.placeholderSet = true;
             view.setValue(placeholderText);
             dom.addClass(view.element, CLASS_NAME);