Use MobileFrontend check for loading DesktopArticleTarget.init

Bug: T236944
Change-Id: I3fd41d0fbb3087d4052db1cb1537e289defa737e
This commit is contained in:
Ed Sanders 2019-11-01 22:05:59 +00:00
parent ffa3742ce1
commit 76d0f02116

View file

@ -62,11 +62,17 @@ class VisualEditorHooks {
* @param Skin $skin The skin that's going to build the UI.
*/
public static function onBeforePageDisplay( OutputPage $output, Skin $skin ) {
$output->addModules( [
'ext.visualEditor.desktopArticleTarget.init',
'ext.visualEditor.targetLoader'
] );
$output->addModuleStyles( [ 'ext.visualEditor.desktopArticleTarget.noscript' ] );
if ( !(
ExtensionRegistry::getInstance()->isLoaded( 'MobileFrontend' ) &&
MediaWikiServices::getInstance()->getService( 'MobileFrontend.Context' )
->shouldDisplayMobileView()
) ) {
$output->addModules( [
'ext.visualEditor.desktopArticleTarget.init',
'ext.visualEditor.targetLoader'
] );
$output->addModuleStyles( [ 'ext.visualEditor.desktopArticleTarget.noscript' ] );
}
// add scroll offset js variable to output
$veConfig = MediaWikiServices::getInstance()->getConfigFactory()
->makeConfig( 'visualeditor' );