Jforrester has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/404597 )

Change subject: OOUI: Manually pull in fix from I99caad7b ahead of the release 
cycle
......................................................................

OOUI: Manually pull in fix from I99caad7b ahead of the release cycle

The .map file will be wrong, but those aren't used, so oh well.

Change-Id: Ic9a6a20541b2f5f1268dd1cf48f767877d2781d4
---
M lib/oojs-ui/oojs-ui-toolbars.js
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/VisualEditor/VisualEditor 
refs/changes/97/404597/1

diff --git a/lib/oojs-ui/oojs-ui-toolbars.js b/lib/oojs-ui/oojs-ui-toolbars.js
index 59d6474..1b57f09 100644
--- a/lib/oojs-ui/oojs-ui-toolbars.js
+++ b/lib/oojs-ui/oojs-ui-toolbars.js
@@ -1978,7 +1978,7 @@
                        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 @@
                                        .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 ?

-- 
To view, visit https://gerrit.wikimedia.org/r/404597
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic9a6a20541b2f5f1268dd1cf48f767877d2781d4
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Jforrester <jforres...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to