jenkins-bot has submitted this change and it was merged.

Change subject: Initialize mobile context as visible
......................................................................


Initialize mobile context as visible

Follow up to I2662df8.

Mobile context hides with a class for transitions so make sure it
is never display: none.

Change-Id: I71fa4a27e6695758eca25c7a89dc532b61aedd72
---
M src/ui/ve.ui.MobileContext.js
1 file changed, 3 insertions(+), 1 deletion(-)

Approvals:
  Catrope: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/src/ui/ve.ui.MobileContext.js b/src/ui/ve.ui.MobileContext.js
index 32b2f2c..df0143d 100644
--- a/src/ui/ve.ui.MobileContext.js
+++ b/src/ui/ve.ui.MobileContext.js
@@ -30,7 +30,9 @@
        // Initialization
        this.$element
                .addClass( 've-ui-mobileContext' )
-               .append( this.menu.$element );
+               .append( this.menu.$element )
+               // Mobile context uses a class to toggle visibility
+               .show();
        this.menu.$element.addClass( 've-ui-mobileContext-menu' );
        this.inspectors.$element.addClass( 've-ui-mobileContext-inspectors' );
        this.surface.getGlobalOverlay().$element.append( 
this.inspectors.$element );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I71fa4a27e6695758eca25c7a89dc532b61aedd72
Gerrit-PatchSet: 1
Gerrit-Project: VisualEditor/VisualEditor
Gerrit-Branch: wmf/1.25wmf7
Gerrit-Owner: Jforrester <[email protected]>
Gerrit-Reviewer: Catrope <[email protected]>
Gerrit-Reviewer: Esanders <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to