fix a img popover bug in firefox

Louis-xhliu · Oct 16, 2014 · 6b4e673 · 6b4e673
1 parent 2951109
commit 6b4e673
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 9 deletions.
diff --git a/lib/simditor.js b/lib/simditor.js
@@ -3707,9 +3707,9 @@ ImageButton = (function(_super) {
         range = document.createRange();
         range.selectNode($img[0]);
         _this.editor.selection.selectRange(range);
-        setTimeout(function() {
-          return _this.editor.body.focus();
-        }, 0);
+        if (!_this.editor.util.supportSelectionChange) {
+          _this.editor.trigger('selectionchanged');
+        }
         return false;
       };
     })(this));

diff --git a/site/assets/scripts/simditor.js b/site/assets/scripts/simditor.js
@@ -3707,9 +3707,9 @@ ImageButton = (function(_super) {
         range = document.createRange();
         range.selectNode($img[0]);
         _this.editor.selection.selectRange(range);
-        setTimeout(function() {
-          return _this.editor.body.focus();
-        }, 0);
+        if (!_this.editor.util.supportSelectionChange) {
+          _this.editor.trigger('selectionchanged');
+        }
         return false;
       };
     })(this));

diff --git a/src/buttons/image.coffee b/src/buttons/image.coffee
@@ -42,9 +42,7 @@ class ImageButton extends Button
       range = document.createRange()
       range.selectNode $img[0]
       @editor.selection.selectRange range
-      setTimeout =>
-        @editor.body.focus()
-      , 0
+      @editor.trigger 'selectionchanged' unless @editor.util.supportSelectionChange
 
       false