resourceloader: Allow mobile target by default on SkinModule
authorjdlrobson <jdlrobson@gmail.com>
Fri, 30 Jun 2017 19:27:04 +0000 (12:27 -0700)
committerKrinkle <krinklemail@gmail.com>
Fri, 30 Jun 2017 21:49:13 +0000 (21:49 +0000)
If a skin is using this class, it's likely to be pretty new.
The targets system was mostly created for older code.

Let's make this the default so skins don't need to do anything
additional to work on mobile.

This simple change makes the Timeless skin work on mobile
when MobileFrontend is installed: ?useformat=mobile&useskin=timeless
It looks beautiful :)

Change-Id: I2ab8a1a634bdc0b5b2084d227c7388b5382e93e8

includes/resourceloader/ResourceLoaderSkinModule.php

index 5740925..1967a95 100644 (file)
  */
 
 class ResourceLoaderSkinModule extends ResourceLoaderFileModule {
  */
 
 class ResourceLoaderSkinModule extends ResourceLoaderFileModule {
+       /**
+        * All skins are assumed to be compatible with mobile
+        */
+       public $targets = [ 'desktop', 'mobile' ];
 
        /**
         * @param ResourceLoaderContext $context
 
        /**
         * @param ResourceLoaderContext $context