From 7d41c8ce062bf46e4f67a25c010297546c6b52f5 Mon Sep 17 00:00:00 2001 From: Aman Agrawal Date: Sat, 11 May 2024 11:32:42 +0000 Subject: [PATCH] narrow: Add minor comment. --- web/src/narrow.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/web/src/narrow.js b/web/src/narrow.js index a278fc88b6..c9567b1175 100644 --- a/web/src/narrow.js +++ b/web/src/narrow.js @@ -475,7 +475,11 @@ export function activate(raw_terms, opts) { // This needs to be called at the same time as updating the // current message list so that we don't need to think about // bugs related to the URL fragment/hash being desynced from - // mesasge_lists.current. + // message_lists.current. + // + // It's fine for the hash change to happen anytime before updating + // the current message list as we are trying to emulate the `hashchange` + // workflow we have which calls `narrow.activate` after hash is updated. if (opts.change_hash) { update_hash_to_match_filter(filter, opts.trigger); }