Changeset 7448


Ignore:
Timestamp:
04/05/12 08:07:23 (3 years ago)
Author:
garry.yao
Message:

#8855: Avoid using window.resizeTo which is broken on Chrome 18.

Location:
CKEditor/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • CKEditor/trunk/CHANGES.html

    r7439 r7448  
    8686                <li><a href="http://dev.ckeditor.com/ticket/8400">#8400</a> : [IE] Fix script error when closing cellProperties dialog if table cell text is selected.</li> 
    8787                <li><a href="http://dev.ckeditor.com/ticket/8248">#8248</a> : [IE8] Fix BACKSPACE/DEL keys when working at the start/end of list items.</li> 
     88                <li><a href="http://dev.ckeditor.com/ticket/8855">#8855</a> : [Chrome] Fix popup window size in Chrome 18.</li> 
    8889        </ul> 
    8990        <h3> 
  • CKEditor/trunk/_source/plugins/popup/plugin.js

    r6348 r7448  
    5050                try 
    5151                { 
    52                         popupWindow.moveTo( left, top ); 
    53                         popupWindow.resizeTo( width, height ); 
     52                        // Chrome 18 is problematic, but it's not really needed here (#8855). 
     53                        var ua = navigator.userAgent.toLowerCase(); 
     54                        if ( ua.indexOf( ' chrome/18' ) == -1 ) 
     55                        { 
     56                                popupWindow.moveTo( left, top ); 
     57                                popupWindow.resizeTo( width, height ); 
     58                        } 
    5459                        popupWindow.focus(); 
    5560                        popupWindow.location.href = url; 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy