Page MenuHomePhabricator

Merge UniversalLanguageSelector ResourceLoader modules to reduce module count
Open, Needs TriagePublic

Description

The following merges can be done to reduce modules:

  • combine jquery.webfonts and ext.uls.webfonts.repository, always loaded together (and then remove the ext.uls.webfonts.fonts module because all it does is depend on those two (now that combined one) and has none of its own anything)
  • combine ext.uls.i18n and ext.uls.messages by having the messages be set via a package file instead of the custom ResourceLoaderULSJsonMessageModule

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change 731135 had a related patch set uploaded (by DannyS712; author: DannyS712):

[mediawiki/extensions/UniversalLanguageSelector@master] Merge jquery.webfonts into ext.uls.webfonts.repository

https://gerrit.wikimedia.org/r/731135

Change 731135 merged by jenkins-bot:

[mediawiki/extensions/UniversalLanguageSelector@master] Merge jquery.webfonts into ext.uls.webfonts.repository

https://gerrit.wikimedia.org/r/731135

I'm sorry that I haven't handled this task. I recently returned from a long bout of unexpected inactivity, and while I plan to resume my contributions here on Phabricator its unfair to claim tasks that I might not work on when others may be interested in handling them. I'm removing myself as the assignee in a batch-action, but if someone feels that I really should be the one to handle this task feel free to re-assign me and I'll take a look.