Changeset 1259 for trunk/modules


Ignore:
Timestamp:
05/10/10 13:19:05 (9 years ago)
Author:
gogo
Message:

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

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/modules/InternetExplorer/InternetExplorer.js

    r1233 r1259  
    334334  { 
    335335    sel = this.getSelection(); 
    336   } 
    337   var range = this.createRange(sel); 
     336  }  
     337  var range = this.createRange(sel);  
    338338  switch ( sel.type ) 
    339339  { 
     
    800800{ 
    801801  if (!sel) sel = this.getSelection(); 
    802   if(sel.type == 'None') this.focusEditor(); 
     802   
     803  // ticket:1508 - when you do a key event within a  
     804  // absolute position div, in IE, the toolbar update 
     805  // for formatblock etc causes a getParentElement() (above) 
     806  // which produces a "None" select, then if we focusEditor() it 
     807  // defocuses the absolute div and focuses into the iframe outside of the 
     808  // div somewhere.   
     809  // 
     810  // Removing this is probably a workaround and maybe it breaks something else 
     811  // focusEditor is used in a number of spots, I woudl have thought it should 
     812  // do nothing if the editor is already focused. 
     813  // 
     814  // if(sel.type == 'None') this.focusEditor(); 
     815   
    803816  return sel.createRange(); 
    804817}; 
Note: See TracChangeset for help on using the changeset viewer.