Issue #3238915 by Bhanu951, hooroomoo, bnjmnm: Refactor (if feasible) uses of...

Issue #3238915 by Bhanu951, hooroomoo, bnjmnm: Refactor (if feasible) uses of the jQuery ready function to use VanillaJS

Issue #3238915 by Bhanu951, hooroomoo, bnjmnm: Refactor (if feasible) uses of...
e9abdb6f · Ben Mullins · 7717f3c5 · e9abdb6f · e9abdb6f
Commit e9abdb6f authored 1 year ago by Ben Mullins
--- a/core/.eslintrc.jquery.json
+++ b/core/.eslintrc.jquery.json
@@ -38,7 +38,7 @@
    "jquery/no-parse-html": 0,
    "jquery/no-prop": 0,
    "jquery/no-proxy": 0,
-    "jquery/no-ready": 0,
+    "jquery/no-ready": 2,
    "jquery/no-serialize": 0,
    "jquery/no-show": 0,
    "jquery/no-size": 2,

--- a/core/modules/statistics/statistics.js
+++ b/core/modules/statistics/statistics.js
@@ -3,8 +3,8 @@
 * Statistics functionality.
 */
-(function ($, Drupal, drupalSettings) {
+(function ($, drupalSettings) {
-  $(document).ready(() => {
+  setTimeout(() => {
    $.ajax({
      type: 'POST',
      cache: false,
@@ -12,4 +12,4 @@
      data: drupalSettings.statistics.data,
    });
  });
-})(jQuery, Drupal, drupalSettings);
+})(jQuery, drupalSettings);