Kaldari has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/54880


Change subject: (bug 46367) fixing display of filter flyout
......................................................................

(bug 46367) fixing display of filter flyout

Doing this in a way that makes more sense and is less hackish/fragile

Change-Id: I3340b5bafe1744585ee4d500f04fc5fb250dbab0
---
M modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.css
M modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.js
2 files changed, 1 insertion(+), 7 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/PageTriage 
refs/changes/80/54880/1

diff --git 
a/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.css 
b/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.css
index f5b83fe..fad3e87 100644
--- a/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.css
+++ b/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.css
@@ -63,7 +63,7 @@
        position: absolute;
        z-index: 51;
        top: 13px;
-       left: 50px;
+       right: -6px;
        visibility: hidden;
        cursor: auto;
 }
diff --git a/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.js 
b/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.js
index bd41363..423c678 100644
--- a/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.js
+++ b/modules/ext.pageTriage.views.list/ext.pageTriage.listControlNav.js
@@ -44,12 +44,6 @@
                        // render and return the template. fill with the 
current model.
                        $( "#mwe-pt-list-control-nav-content").html( 
this.template() );
 
-                       // align the filter dropdown box with the dropdown 
control widget
-                       var startSide = $( "body" ).hasClass( "rtl" ) ? "right" 
: "left";
-                       var newStart = $( '#mwe-pt-filter-dropdown-control' 
).width() - 20;
-                       $( "#mwe-pt-control-dropdown" ).css( startSide, 
newStart );
-                       $( "#mwe-pt-control-dropdown-pokey" ).css( startSide, 
newStart + 5 );
-
                        //
                        // now that the template's been inserted, set up some 
events for controlling it
                        //

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3340b5bafe1744585ee4d500f04fc5fb250dbab0
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/PageTriage
Gerrit-Branch: master
Gerrit-Owner: Kaldari <rkald...@wikimedia.org>

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

Reply via email to