From b60fab333106af4e442157f8e7939962e3d07062 Mon Sep 17 00:00:00 2001 From: Stephen Niedzielski Date: Mon, 25 Jun 2018 14:07:29 -0500 Subject: [PATCH] Fix: add login icon as a server render dependency of main menu The profile icon is necessary for logged in users. This is known when parsing skin.json but was not added in MinervaHooks.php. This patch adds the dependency unconditionally although it is only needed for the server render of Special:MobileMenu for logged in users. Bug: T196893 Change-Id: I2a8485299162c01d28ca1b07e7071ad0c0ef44fc --- includes/MinervaHooks.php | 1 + 1 file changed, 1 insertion(+) diff --git a/includes/MinervaHooks.php b/includes/MinervaHooks.php index 43a254c..9c6f654 100644 --- a/includes/MinervaHooks.php +++ b/includes/MinervaHooks.php @@ -118,6 +118,7 @@ class MinervaHooks { switch ( $name ) { case 'MobileMenu': $out->addModuleStyles( [ + 'skins.minerva.icons.images.variants', 'skins.minerva.mainMenu.icons', 'skins.minerva.mainMenu.styles', ] );