Merge pull request #121 from teambition/master

fix selection bug for IE
sofish · Nov 21, 2014 · 0d940bc · 0d940bc
2 parents 779d67a + e62daef
commit 0d940bc
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/pen.js b/src/pen.js
@@ -273,6 +273,7 @@
 
     addListener(ctx, editor, 'blur', function() {
       checkPlaceholder(ctx);
+      ctx.checkContentChange();
     });
 
     // listen for paste and clear style
@@ -507,8 +508,10 @@
       range = this.getRange();
       range.collapse(false); // set to end
     }
-    selection.removeAllRanges();
-    selection.addRange(range);
+    try {
+      selection.removeAllRanges();
+      selection.addRange(range);
+    } catch (e) {/* IE throws error sometimes*/}
     return this;
   };
 
@@ -696,8 +699,10 @@
 
     if (!isAJoke) {
       removeAllListeners(this);
-      selection.removeAllRanges();
-      this._menu.parentNode.removeChild(this._menu);
+      try {
+        selection.removeAllRanges();
+        this._menu.parentNode.removeChild(this._menu);
+      } catch (e) {/* IE throws error sometimes*/}
     } else {
       initToolbar(this);
       initEvents(this);