Ticket #8629: 8629.patch

_source/plugins/floatpanel/plugin.js

@@ -247 +247 @@
                                                         else
                                                                 target.removeStyle( 'height' );
 
+                                                        // Place for measurement.
+                                                        element.setStyle( 'left', CKEDITOR.tools.cssLength( left ) );
+
                                                         var panelElement = panel.element,
                                                                 panelWindow = panelElement.getWindow(),
-                                                                windowScroll = panelWindow.getScrollPosition(),
-                                                                viewportSize = panelWindow.getViewPaneSize(),
-                                                                panelSize =
-                                                                {
-                                                                        'height' : panelElement.$.offsetHeight,
-                                                                        'width' : panelElement.$.offsetWidth
-                                                                };
+                                                                rect = element.$.getBoundingClientRect(),
+                                                                viewportSize = panelWindow.getViewPaneSize();
 
-                                                        // If the menu is horizontal off, shift it toward
-                                                        // the opposite language direction.
-                                                        if ( rtl ? left < 0 : left + panelSize.width > viewportSize.width + windowScroll.x )
-                                                                left += ( panelSize.width * ( rtl ? 1 : -1 ) );
+                                                        // Compensation for browsers that dont support "width" and "height".
+                                                        var rectWidth = rect.width || rect.right - rect.left,
+                                                        rectHeight = rect.height || rect.bottom - rect.top;
 
-                                                        // Vertical off screen is simpler.
-                                                        if ( top + panelSize.height > viewportSize.height + windowScroll.y )
-                                                                top -= panelSize.height;
+                                                        // Check to flip the panel display direction to opposite side
+                                                        // when there's not enough space available.
+                                                        var spaceAfter = rtl ? rect.right : viewportSize.width - rect.left,
+                                                                spaceBefore = rtl ? viewportSize.width - rect.right : rect.left;
+
+                                                        // Flip horizontally.
+                                                        if ( spaceAfter - rectWidth < 0 && spaceBefore > spaceAfter )
+                                                                left += ( rectWidth * ( rtl ? 1 : -1 ) );
+
+                                                        // Flip vertically.
+                                                        var spaceBelow = viewportSize.height - rect.top,
+                                                                spaceAbove = rect.top;
+
+                                                        if ( spaceBelow - rectHeight < 0 && spaceAbove > spaceBelow )
+                                                                top -= rectHeight;
 
                                                         // If IE is in R™TL, we have troubles with absolute
                                                         // position and horizontal scrolls. Here we have a