Bartosz Dziewoński has uploaded a new change for review.

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

Change subject: Hide page status indicators even if they aren't :visible
......................................................................

Hide page status indicators even if they aren't :visible

The container is zero-height and zero-width, so jQuery doesn't
consider it :visible, and yet it still takes up visual space.

Bug: 72693
Change-Id: I3bed5394a2a64a9f7f0a331f850aa412b5b11b7d
---
M modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/VisualEditor 
refs/changes/60/169860/1

diff --git a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js 
b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
index ca10ef4..49321d2 100644
--- a/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
+++ b/modules/ve-mw/init/targets/ve.init.mw.ViewPageTarget.js
@@ -1420,7 +1420,7 @@
                .addClass( 've-hide' )
                .slideUp( 'fast' );
        // Hide page status indicators (if present)
-       $( '.mw-indicators:visible' )
+       $( '.mw-indicators' )
                .addClass( 've-hide' )
                .fadeOut( 'fast' );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3bed5394a2a64a9f7f0a331f850aa412b5b11b7d
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/VisualEditor
Gerrit-Branch: master
Gerrit-Owner: Bartosz Dziewoński <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to