Page MenuHomePhabricator

TwoColConflict and RevisionSlider resourceloader modules can be packed into one or two modules to save load time.
Open, Needs TriagePublic

Description

@Ladsgroup noticed that these two extensions are nicely decomposed into a large number of ResourceLoader modules. Unfortunately, this means that every Wikimedia page request includes the names of all of our modules, which adds up to 1.6kB uncompressed and 600B compressed, during the HEAD load which impacts overall page render time. We should either make these modules load conditionally, only when needed, or use something like Webpack to build one or two modules instead. The latter approach is already in use by Wikibase client, so we can learn from their experiences.

Event Timeline

awight created this task.Mon, Aug 5, 9:45 AM
Restricted Application added a project: TCB-Team. · View Herald TranscriptMon, Aug 5, 9:45 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
awight updated the task description. (Show Details)Mon, Aug 5, 9:48 AM

Change 530213 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/RevisionSlider@master] Merge some resource loader modules

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

Change 530334 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/TwoColConflict@master] Merge some resource loader modules

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

Change 530213 merged by jenkins-bot:
[mediawiki/extensions/RevisionSlider@master] Merge some resource loader modules

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

Change 530542 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/TwoColConflict@master] Drop ext.TwoColConflict.Inline.AutoScroll module

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

Change 530543 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/TwoColConflict@master] Drop ext.TwoColConflict.Settings

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

Change 530544 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/TwoColConflict@master] Drop ext.TwoColConflict.Split.Merger

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

Change 530544 abandoned by Ladsgroup:
Drop ext.TwoColConflict.Split.Merger

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

Change 530334 abandoned by Ladsgroup:
Merge some resource loader modules

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