Fomafix has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/330879 )

Change subject: Replace $( document ).ready(...) by $(...)
......................................................................

Replace $( document ).ready(...) by $(...)

In jQuery 3 $(document).ready(...) is deprecated.
https://jquery.com/upgrade-guide/3.0/#deprecated-document-ready-handlers-other-than-jquery-function

Change-Id: I1a9f578d8cf8fa2d9dd38a1b2d3a0b3c32914b74
---
M resources/js/ext.uls.compactlinks.js
M resources/js/ext.uls.interface.js
M resources/js/ext.uls.webfonts.js
M resources/js/ext.uls.webfonts.mobile.js
M tests/webfonts.html
5 files changed, 5 insertions(+), 5 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/UniversalLanguageSelector 
refs/changes/79/330879/1

diff --git a/resources/js/ext.uls.compactlinks.js 
b/resources/js/ext.uls.compactlinks.js
index 6e1b4b0..3c74371 100644
--- a/resources/js/ext.uls.compactlinks.js
+++ b/resources/js/ext.uls.compactlinks.js
@@ -506,7 +506,7 @@
        if ( document.readyState === 'interactive' ) {
                createCompactList();
        } else {
-               $( document ).ready( createCompactList );
+               $( createCompactList );
        }
 
 }( jQuery, mediaWiki ) );
diff --git a/resources/js/ext.uls.interface.js 
b/resources/js/ext.uls.interface.js
index 0d48ccb..0e7d38b 100644
--- a/resources/js/ext.uls.interface.js
+++ b/resources/js/ext.uls.interface.js
@@ -523,6 +523,6 @@
        if ( document.readyState === 'interactive' ) {
                init();
        } else {
-               $( document ).ready( init );
+               $( init );
        }
 }( jQuery, mediaWiki ) );
diff --git a/resources/js/ext.uls.webfonts.js b/resources/js/ext.uls.webfonts.js
index a28530d..0bf41b9 100644
--- a/resources/js/ext.uls.webfonts.js
+++ b/resources/js/ext.uls.webfonts.js
@@ -224,7 +224,7 @@
                }, 0 );
        };
 
-       $( document ).ready( function () {
+       $( function () {
                mw.webfonts.preferences.load();
 
                if ( mw.webfonts.preferences.isEnabled() ) {
diff --git a/resources/js/ext.uls.webfonts.mobile.js 
b/resources/js/ext.uls.webfonts.mobile.js
index 0702b30..576d617 100644
--- a/resources/js/ext.uls.webfonts.mobile.js
+++ b/resources/js/ext.uls.webfonts.mobile.js
@@ -28,7 +28,7 @@
        mediawikiFontRepository.base = mw.config.get( 'wgExtensionAssetsPath' ) 
+
                '/UniversalLanguageSelector/data/fontrepo/fonts/';
 
-       $( document ).ready( function () {
+       $( function () {
                // MediaWiki specific overrides for jquery.webfonts
                $.extend( $.fn.webfonts.defaults, {
                        repository: mediawikiFontRepository,
diff --git a/tests/webfonts.html b/tests/webfonts.html
index db276d8..0a75645 100644
--- a/tests/webfonts.html
+++ b/tests/webfonts.html
@@ -11,7 +11,7 @@
        <script src="../lib/jquery.uls/src/jquery.uls.data.utils.js"></script>
        <script src="../resources/js/ext.uls.webfonts.repository.js"></script>
        <script>
-               $( document ).ready( function () {
+               $( function () {
                        var $webfonts, fonts, languages, $fontSelector, 
$langselector;
 
                        $( 'div#webfonts-preview-area' ).webfonts( {

-- 
To view, visit https://gerrit.wikimedia.org/r/330879
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1a9f578d8cf8fa2d9dd38a1b2d3a0b3c32914b74
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/UniversalLanguageSelector
Gerrit-Branch: master
Gerrit-Owner: Fomafix <[email protected]>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to