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

Change subject: Update mobile target check
......................................................................

Update mobile target check

Change-Id: I1488658aa5d0bcbccf5cc011bcf5da5126016de9
---
M modules/editor/editors/visualeditor/ext.flow.editors.visualeditor.js
M modules/flow/ui/widgets/editor/editors/mw.flow.ui.VisualEditorWidget.js
2 files changed, 2 insertions(+), 6 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Flow 
refs/changes/10/390310/1

diff --git 
a/modules/editor/editors/visualeditor/ext.flow.editors.visualeditor.js 
b/modules/editor/editors/visualeditor/ext.flow.editors.visualeditor.js
index 2bff44d..b77656f 100644
--- a/modules/editor/editors/visualeditor/ext.flow.editors.visualeditor.js
+++ b/modules/editor/editors/visualeditor/ext.flow.editors.visualeditor.js
@@ -202,11 +202,9 @@
        // Static methods
 
        mw.flow.editors.visualeditor.static.isSupported = function () {
-               var isMobileTarget = ( mw.config.get( 'skin' ) === 'minerva' );
-
                /* global VisualEditorSupportCheck */
                return !!(
-                       !isMobileTarget &&
+                       !OO.ui.isMobile() &&
                        mw.loader.getState( 'ext.visualEditor.core' ) &&
                        mw.config.get( 'wgFlowEditorList' ).indexOf( 
'visualeditor' ) !== -1 &&
                        window.VisualEditorSupportCheck && 
VisualEditorSupportCheck()
diff --git 
a/modules/flow/ui/widgets/editor/editors/mw.flow.ui.VisualEditorWidget.js 
b/modules/flow/ui/widgets/editor/editors/mw.flow.ui.VisualEditorWidget.js
index 4f42383..3311c8c 100644
--- a/modules/flow/ui/widgets/editor/editors/mw.flow.ui.VisualEditorWidget.js
+++ b/modules/flow/ui/widgets/editor/editors/mw.flow.ui.VisualEditorWidget.js
@@ -40,11 +40,9 @@
         * @inheritdoc
         */
        mw.flow.ui.VisualEditorWidget.static.isSupported = function () {
-               var isMobileTarget = ( mw.config.get( 'skin' ) === 'minerva' );
-
                /* global VisualEditorSupportCheck */
                return !!(
-                       !isMobileTarget &&
+                       !OO.ui.isMobile() &&
                        mw.loader.getState( 'ext.visualEditor.core' ) &&
                        mw.config.get( 'wgFlowEditorList' ).indexOf( 
'visualeditor' ) !== -1 &&
                        window.VisualEditorSupportCheck && 
VisualEditorSupportCheck()

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1488658aa5d0bcbccf5cc011bcf5da5126016de9
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Flow
Gerrit-Branch: master
Gerrit-Owner: Esanders <[email protected]>

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

Reply via email to