Changeset 5929


Ignore:
Timestamp:
10/11/10 19:08:41 (4 years ago)
Author:
Saare
Message:

Post-fixing #6341: [FF] Cursor below text doesn't get the text icon.

Location:
CKEditor/trunk/_source/plugins
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/_source/plugins/autogrow/plugin.js

    r5924 r5929  
    1515 
    1616                // We can not use documentElement to calculate the height for IE (#6061). 
    17                 // It is not good for Quirks, yet using offsetHeight would also not work as expected (#6408). 
    18                 if ( CKEDITOR.env.ie ) 
    19                         newHeight = doc.getBody().$.scrollHeight + ( CKEDITOR.env.quirks ? 0 : 24 ); 
     17                // It is not good for IE Quirks, yet using offsetHeight would also not work as expected (#6408). 
     18                // We do the same for FF because of the html height workaround (#6341). 
     19                if ( CKEDITOR.env.ie || CKEDITOR.env.gecko ) 
     20                        newHeight = doc.getBody().$.scrollHeight + ( CKEDITOR.env.ie && CKEDITOR.env.quirks ? 0 : 24 ); 
    2021                else 
    2122                        newHeight = doc.getDocumentElement().$.offsetHeight; 
  • CKEditor/trunk/_source/plugins/wysiwygarea/plugin.js

    r5908 r5929  
    543543 
    544544                                                // Gecko/Webkit need some help when selecting control type elements. (#3448) 
    545                                                 if ( !( CKEDITOR.env.ie || CKEDITOR.env.opera) ) 
     545                                                if ( !( CKEDITOR.env.ie || CKEDITOR.env.opera ) ) 
    546546                                                { 
    547547                                                        domDocument.on( 'mousedown', function( ev ) 
     
    555555                                                if ( CKEDITOR.env.gecko ) 
    556556                                                { 
     557                                                        // Set the HTML style to 100% to have the text cursor in affect (#6341) 
     558                                                        domDocument.getDocumentElement().setStyle( 'height', '100%' ); 
    557559                                                        domDocument.on( 'mouseup', function( ev ) 
    558560                                                        { 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy