From bc7bec9a95e374ba10d3000b899f97861886ec3b Mon Sep 17 00:00:00 2001
From: Anders Kaseorg <anders@zulip.com>
Date: Wed, 22 Jul 2020 12:40:28 -0700
Subject: [PATCH] landing-page: Clean up ready callbacks.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

jQuery’s $(callback) already checks document.readyState to decide
whether to run the callback immediately (that’s like, jQuery’s entire
value proposition).  We probably don’t need ready callbacks at all
anymore thanks to <script defer>, but that’s a larger change.

Signed-off-by: Anders Kaseorg <anders@zulip.com>
---
 static/js/portico/landing-page.js | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/static/js/portico/landing-page.js b/static/js/portico/landing-page.js
index 099148b14c..e98c112d67 100644
--- a/static/js/portico/landing-page.js
+++ b/static/js/portico/landing-page.js
@@ -201,8 +201,7 @@ const events = function () {
     }
 };
 
-// run this callback when the page is determined to have loaded.
-const load = function () {
+$(() => {
     // Initiate the bootstrap carousel logic
     $(".carousel").carousel({
         interval: false,
@@ -238,15 +237,7 @@ const load = function () {
 
     // Set up events / categories / search
     events();
-};
 
-if (document.readyState === "complete") {
-    load();
-} else {
-    $(load);
-}
-
-$(() => {
     if (window.location.pathname === "/team/") {
         render_tabs();
     }