Display result box only after it's been populated

geigi · Dec 3, 2023 · 3e70c78 · 3e70c78
1 parent 33944e4
commit 3e70c78
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/cozy/ui/search_view.py b/cozy/ui/search_view.py
@@ -118,10 +118,10 @@ def _populate_listbox(
         if not results:
             return
 
-        box.set_visible(True)
-
         for result in results:
             if self.search_thread_stop.is_set():
                 return
 
             listbox.append(row_type(result, callback))
+
+        box.set_visible(True)
diff --git a/cozy/view_model/search_view_model.py b/cozy/view_model/search_view_model.py
@@ -50,7 +50,7 @@ def readers(self):
 
         return split_strings_to_set(readers)
 
-    def search(self, search_query: str, callback: Callable[[], None], thread_event):
+    def search(self, search_query: str, callback: Callable[[list[Book], list[str], list[str]], None], thread_event):
         search_query = search_query.lower()
 
         # We need the main context to call methods in the main thread after the search is finished