Changeset 1259

Timestamp:

05/10/10 13:19:05 (3 years ago)

Author:

gogo

Message:

ticket:1508
Potential fix for cursor problems in absolute positioned div in IE.

Files:

• trunk/modules/InternetExplorer/InternetExplorer.js (modified) (2 diffs)

trunk/modules/InternetExplorer/InternetExplorer.js

@@ -334 +334 @@
   {
     sel = this.getSelection();
-  }
-  var range = this.createRange(sel);
+  } 
+  var range = this.createRange(sel); 
   switch ( sel.type )
   {
@@ -800 +800 @@
 {
   if (!sel) sel = this.getSelection();
-  if(sel.type == 'None') this.focusEditor();
+  
+  // ticket:1508 - when you do a key event within a 
+  // absolute position div, in IE, the toolbar update
+  // for formatblock etc causes a getParentElement() (above)
+  // which produces a "None" select, then if we focusEditor() it
+  // defocuses the absolute div and focuses into the iframe outside of the
+  // div somewhere.  
+  //
+  // Removing this is probably a workaround and maybe it breaks something else
+  // focusEditor is used in a number of spots, I woudl have thought it should
+  // do nothing if the editor is already focused.
+  //
+  // if(sel.type == 'None') this.focusEditor();
+  
   return sel.createRange();
 };