Merge jquery.webfonts into ext.uls.webfonts.repository
The two modules are always loaded together and don't need separate entry points. Additionally, this makes the ext.uls.webfonts.fonts just depend on the ext.uls.webfonts.repository module and not do anything else, so remove it in favor of referencing ext.uls.webfonts.repository directly where needed. Bug: T292042 Change-Id: I067b9eda923fcb0d5da0ed80030356f1f747259b
This commit is contained in:
1 parent
a01bf88f9e
commit
e17f5c0b39
3 files changed
+6
-19
No files matched your search
@@ -122,7 +122,7 @@
|
||||
*/
|
||||
setupWebFonts: function () {
|
||||
var d = $.Deferred();
|
||||
mw.loader.using( [ 'ext.uls.webfonts.fonts' ] ).then( function () {
|
||||
mw.loader.using( [ 'ext.uls.webfonts.repository' ] ).then( function () {
|
||||
if ( this.isWebFontsEnabled ) {
|
||||
mw.webfonts.setup();
|
||||
}
|
||||
|
||||
@@ -137,7 +137,7 @@
|
||||
if ( mw.webfonts.preferences.isEnabled() ) {
|
||||
// Queue to next idle period to optimize loading.
|
||||
mw.requestIdleCallback( function () {
|
||||
mw.loader.using( 'ext.uls.webfonts.fonts' ).then( mw.webfonts.setup );
|
||||
mw.loader.using( 'ext.uls.webfonts.repository' ).then( mw.webfonts.setup );
|
||||
} );
|
||||
}
|
||||
} );
|
||||
|
||||
Reference in new issue
Block a user