From 419df9d44673bf1abe3cda7ba6cafee9dd6eaba4 Mon Sep 17 00:00:00 2001
From: Shpuld Shpuldson <shp@cock.li>
Date: Thu, 19 Nov 2020 12:35:21 +0200
Subject: update some documentation

---
 src/components/emoji_input/suggestor.js | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

(limited to 'src/components/emoji_input/suggestor.js')

diff --git a/src/components/emoji_input/suggestor.js b/src/components/emoji_input/suggestor.js
index 0d268f85..14a2b41e 100644
--- a/src/components/emoji_input/suggestor.js
+++ b/src/components/emoji_input/suggestor.js
@@ -57,6 +57,8 @@ export const suggestEmoji = emojis => input => {
 }
 
 export const suggestUsers = ({ dispatch, state }) => {
+  // Keep some persistent values in closure, most importantly for the
+  // custom debounce to work. Lodash debounce does not return a promise.
   let suggestions = []
   let previousQuery = ''
   let timeout = null
@@ -66,7 +68,6 @@ export const suggestUsers = ({ dispatch, state }) => {
   const debounceUserSearch = (query) => {
     cancelUserSearch && cancelUserSearch()
     return new Promise((resolve, reject) => {
-      clearTimeout(timeout)
       timeout = setTimeout(() => {
         userSearch(query).then(resolve).catch(reject)
       }, 300)
@@ -95,11 +96,6 @@ export const suggestUsers = ({ dispatch, state }) => {
       user =>
         user.screen_name.toLowerCase().startsWith(noPrefix) ||
         user.name.toLowerCase().startsWith(noPrefix)
-
-      /* taking only 20 results so that sorting is a bit cheaper, we display
-       * only 5 anyway. could be inaccurate, but we ideally we should query
-       * backend anyway
-       */
     ).slice(0, 20).sort((a, b) => {
       let aScore = 0
       let bScore = 0
-- 
cgit v1.2.3-70-g09d2