From cbb46f74514e51624cde25f8dcf2fab8121ff1b8 Mon Sep 17 00:00:00 2001
From: Santhosh Thottingal <santhosh.thottingal@gmail.com>
Date: Tue, 17 May 2016 11:40:15 +0530
Subject: [PATCH] Compact language links: Delay loading of jquery.uls module

Load jquery.uls only when the trigger is clicked.
Reduces the initial footprint by 63.9 KB (uncompressed)

Bug: T135210
Change-Id: I972cb6ea1794485917ea530f7abe68bdddc2014f
---
 extension.json                       | 1 -
 resources/js/ext.uls.compactlinks.js | 9 ++++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/extension.json b/extension.json
index 7fb2aa51..101d1f4f 100644
--- a/extension.json
+++ b/extension.json
@@ -242,7 +242,6 @@
 			"styles": "css/ext.uls.compactlinks.less",
 			"dependencies": [
 				"es5-shim",
-				"jquery.uls.compact",
 				"mediawiki.language",
 				"mediawiki.ui.button"
 			],
diff --git a/resources/js/ext.uls.compactlinks.js b/resources/js/ext.uls.compactlinks.js
index e1e6c3af..5fddbbb8 100644
--- a/resources/js/ext.uls.compactlinks.js
+++ b/resources/js/ext.uls.compactlinks.js
@@ -69,7 +69,7 @@
 				return;
 			}
 
-			mw.loader.using( 'ext.uls.mediawiki' ).done( function () {
+			mw.loader.using( 'ext.uls.init' ).done( function () {
 				// If we're only a bit beyond max, limit to 7 instead of 9.
 				// FIXME: This assumes the max is 9.
 				self.compactSize = ( self.listSize <= 12 ) ? 7 : max;
@@ -187,8 +187,11 @@
 			var self = this;
 
 			this.$trigger.one( 'click', function () {
-				self.createSelector( self.$trigger );
-				self.$trigger.click();
+				// Load the ULS now.
+				mw.loader.using( 'ext.uls.mediawiki' ).done( function () {
+					self.createSelector( self.$trigger );
+					self.$trigger.click();
+				} );
 			} );
 		},