From 20a8077cbd4067a75a170ea20064fd66ad567435 Mon Sep 17 00:00:00 2001
From: Steve Howell <showell@zulip.com>
Date: Tue, 16 Feb 2021 12:50:09 +0000
Subject: [PATCH] minor: Make sort_toggler a const.

Unlike the other toggler in this module, there
is no reason that sort_toggler has to be a module
level variable.
---
 static/js/subs.js | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/static/js/subs.js b/static/js/subs.js
index 6f1e2e71f6..df89748535 100644
--- a/static/js/subs.js
+++ b/static/js/subs.js
@@ -529,7 +529,6 @@ exports.switch_stream_tab = function (tab_name) {
     stream_edit.setup_subscriptions_tab_hash(tab_name);
 };
 
-exports.sort_toggler = undefined;
 exports.switch_stream_sort = function (tab_name) {
     if (
         tab_name === "by-stream-name" ||
@@ -560,7 +559,7 @@ exports.setup_page = function (callback) {
     function initialize_components() {
         // Sort by name by default when opening "Manage streams".
         sort_order = "by-stream-name";
-        exports.sort_toggler = components.toggle({
+        const sort_toggler = components.toggle({
             values: [
                 {
                     label_html: `<i class="fa fa-sort-alpha-asc" title="${i18n.t(
@@ -586,7 +585,7 @@ exports.setup_page = function (callback) {
                 exports.switch_stream_sort(key);
             },
         });
-        $("#subscriptions_table .search-container").prepend(exports.sort_toggler.get());
+        $("#subscriptions_table .search-container").prepend(sort_toggler.get());
 
         // Reset our internal state to reflect that we're initially in
         // the "Subscribed" tab if we're reopening "Manage streams".