From e7ee5c20c6ba30c925a83c956b6411c3a25e915b Mon Sep 17 00:00:00 2001 From: Ed Sanders Date: Mon, 4 Jan 2016 10:43:43 -0800 Subject: [PATCH] Education popups: Only show for DesktopArticleTarget For now they are not well supported by other targets (e.g. Flow and Mobile). Change-Id: Ibd1f3c8994611661b52c0cf36438b41f623b9451 --- modules/ve-mw/ui/tools/ve.ui.MWEducationPopupTool.js | 1 + 1 file changed, 1 insertion(+) diff --git a/modules/ve-mw/ui/tools/ve.ui.MWEducationPopupTool.js b/modules/ve-mw/ui/tools/ve.ui.MWEducationPopupTool.js index ad9734a672..2bb978301c 100644 --- a/modules/ve-mw/ui/tools/ve.ui.MWEducationPopupTool.js +++ b/modules/ve-mw/ui/tools/ve.ui.MWEducationPopupTool.js @@ -23,6 +23,7 @@ ve.ui.MWEducationPopupTool = function VeUiMwEducationPopupTool( config ) { config = config || {}; if ( + !( ve.init.mw.DesktopArticleTarget && ve.init.target instanceof ve.init.mw.DesktopArticleTarget ) || ve.init.target.dummyToolbar || ( !prefSaysShow &&