This is follow-up to work done for T428220.
We want to reduce the number of modules loaded for icons in SkinMinerva.php. Having multiple modules is bad for front-end performance. Here's a related gerrit comment. https://gerrit.wikimedia.org/r/c/mediawiki/skins/MinervaNeue/+/1306430/comment/5c20b2eb_e7fe2ef4/
| Subject | Author | Repo | Branch | Lines +/- | |
|---|---|---|---|---|---|
| Icons Modules: Merge skins.minerva.overflow.icons into skins.minerva.icons | EMcFarland | mediawiki/skins/MinervaNeue | master | +5 -19 |
Change #1309687 had a related patch set uploaded (by EMcFarland; author: EMcFarland):
[mediawiki/skins/MinervaNeue@master] Icons Modules: Merge skins.minerva.overflow.icons into skins.minerva.icons
@EMcFarland-WMF: Assuming this task is about the MinervaNeue code project, hence adding that project tag so other people who don't know or don't care about team tags can also find this task when searching via projects or looking at workboards. Please set appropriate project tags when possible. Thanks!
Change #1309687 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Icons Modules: Merge skins.minerva.overflow.icons into skins.minerva.icons
I'll resolve based on the fact that it's a config change. Related issues can be filed in a follow up task