Changeset 7448

Timestamp:

04/05/12 08:07:23 (14 months ago)

Author:

garry.yao

Message:

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

Location:

CKEditor/trunk

Files:

• CHANGES.html (modified) (1 diff)
• _source/plugins/popup/plugin.js (modified) (1 diff)

CKEditor/trunk/CHANGES.html

@@ -86 +86 @@
                 <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>
                 <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>
+                <li><a href="http://dev.ckeditor.com/ticket/8855">#8855</a> : [Chrome] Fix popup window size in Chrome 18.</li>
         </ul>
         <h3>

CKEditor/trunk/_source/plugins/popup/plugin.js

@@ -50 +50 @@
                 try
                 {
-                        popupWindow.moveTo( left, top );
-                        popupWindow.resizeTo( width, height );
+                        // Chrome 18 is problematic, but it's not really needed here (#8855).
+                        var ua = navigator.userAgent.toLowerCase();
+                        if ( ua.indexOf( ' chrome/18' ) == -1 )
+                        {
+                                popupWindow.moveTo( left, top );
+                                popupWindow.resizeTo( width, height );
+                        }
                         popupWindow.focus();
                         popupWindow.location.href = url;