OOUI: Manually pull in fix from I99caad7b ahead of the release cycle
authorJames D. Forrester <jforrester@wikimedia.org>
Fri, 12 Jan 2018 16:56:02 +0000 (08:56 -0800)
committerJames D. Forrester <jforrester@wikimedia.org>
Fri, 12 Jan 2018 16:56:02 +0000 (08:56 -0800)
The .map file will be wrong, but those aren't used outside of an
experimental branch yet, so it's not the end of the world.

Bug: T184665
Change-Id: I9ff1ed80496ec61b836de4221dd36cde6d4a71ac

resources/lib/oojs-ui/oojs-ui-toolbars.js

index 59d6474..1b57f09 100644 (file)
@@ -1978,7 +1978,7 @@ OO.ui.PopupToolGroup.prototype.setActive = function ( value ) {
                        this.$element.addClass( 'oo-ui-popupToolGroup-active oo-ui-popupToolGroup-left' );
                        this.setFlags( { progressive: true } );
                        this.toggleClipping( true );
-                       if ( this.isClippedHorizontally() || this.isFloatableOutOfView() ) {
+                       if ( this.isClippedHorizontally() ) {
                                // Anchoring to the left caused the popup to clip, so anchor it to the right instead
                                this.toggleClipping( false );
                                this.$element
@@ -1986,7 +1986,7 @@ OO.ui.PopupToolGroup.prototype.setActive = function ( value ) {
                                        .addClass( 'oo-ui-popupToolGroup-right' );
                                this.toggleClipping( true );
                        }
-                       if ( this.isClippedHorizontally() || this.isFloatableOutOfView() ) {
+                       if ( this.isClippedHorizontally() ) {
                                // Anchoring to the right also caused the popup to clip, so just make it fill the container
                                containerWidth = this.$clippableScrollableContainer.width();
                                containerLeft = this.$clippableScrollableContainer[ 0 ] === document.documentElement ?