Change Details

The ext.uls.interface module is a [[ https://en.wikipedia.org/w/load.php?modules=ext.uls.interface | 3.36kb module ]] that loads on launch of all pages. It accounts for many of the warnings in T235712. It loads if isToolbarEnabled and that function should return false when the skin is Minerva which I believe is the intended behaviour here. = Acceptance criteria [] Note any dependencies of the module will also need to be updated to set appropriate targets - ext.uls.webfonts, ext.uls.common and jquery.uls.data. These don't appear to be added directly to the page so this should not impact the mobile site. [] The module itself should set `"targets": [ "desktop", "mobile" ],` so that it does not throw the warning described in T235712 [] Update the body of isToolbarEnabled to consider skin. isToolbarEnabled should return false in skins such as Minerva where no toolbar is enabled. ``` private static function isToolbarEnabled( User $user ) { ``` [] Create a task to audit the ext.uls.interface module to see if the size of the module can be reduced and any of its functionality deferred. [] ULS should appear when viewing Vector on the mobile domain https://en.m.wikipedia.org/wiki/Spain?useskin=vector

The ext.uls.interface module is a [[ https://en.wikipedia.org/w/load.php?modules=ext.uls.interface | 3.36kb module ]] that loads on launch of all pages. It accounts for many of the warnings in T235712. It loads if isToolbarEnabled and that function should return false when the skin is Minerva which I believe is the intended behaviour here. = Acceptance criteria [] Note any dependencies of the module will also need to be updated to set appropriate targets - ext.uls.webfonts, ext.uls.common and jquery.uls.data. These don't appear to be added directly to the page so this should not impact the mobile site. [] The module itself should set `"targets": [ "desktop", "mobile" ],` so that it does not throw the warning described in T235712 [] Update the body of isToolbarEnabled to consider skin. isToolbarEnabled should return false in skins such as Minerva where no toolbar is enabled. ``` private static function isToolbarEnabled( User $user ) { ``` [] Create a task to audit the ext.uls.interface module to see if the size of the module can be reduced and any of its functionality deferred. [] ULS should appear when viewing Vector on the mobile domain https://en.m.wikipedia.org/wiki/Spain?useskin=vector

The ext.uls.interface module is a [[ https://en.wikipedia.org/w/load.php?modules=ext.uls.interface | 3.36kb module ]] that loads on launch of all pages. It accounts for many of the warnings in T235712. It loads if isToolbarEnabled and that function should return false when the skin is Minerva which I believe is the intended behaviour here. = Acceptance criteria [] Note any dependencies of the module will also need to be updated to set appropriate targets - ext.uls.webfonts, ext.uls.common and jquery.uls.data. These don't appear to be added directly to the page so this should not impact the mobile site. [] The module itself should set `"targets": [ "desktop", "mobile" ],` so that it does not throw the warning described in T235712 [] Update the body of isToolbarEnabled to consider skin. isToolbarEnabled should return false in skins such as Minerva where no toolbar is enabled. ``` private static function isToolbarEnabled( User $user ) { ``` [] Create a task to audit the ext.uls.interface module to see if the size of the module can be reduced and any of its functionality deferred. [] ULS should appear when viewing Vector on the mobile domain https://en.m.wikipedia.org/wiki/Spain?useskin=vector