Skip to content

Commit ecd883b

Browse files
committed
boxes/views: Treat spaces as an empty search in PanelSearchBox.
This amends PanelSearchBox and related update functions to treat spaces as an empty search by using left strip, i.e., the search is canceled and the list is not updated.
1 parent cbccdd7 commit ecd883b

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

zulipterminal/ui_tools/boxes.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1213,7 +1213,8 @@ def reset_search_text(self) -> None:
12131213
self.set_edit_text(self.search_text)
12141214

12151215
def keypress(self, size: urwid_Size, key: str) -> Optional[str]:
1216-
if ((is_command_key('ENTER', key) and self.get_edit_text() == '')
1216+
if ((is_command_key('ENTER', key)
1217+
and self.get_edit_text().lstrip() == '')
12171218
or is_command_key('GO_BACK', key)):
12181219
self.panel_view.view.controller.exit_editor_mode()
12191220
self.reset_search_text()

zulipterminal/ui_tools/views.py

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -283,6 +283,8 @@ def __init__(self, streams_btn_list: List[Any], view: Any) -> None:
283283

284284
@asynch
285285
def update_streams(self, search_box: Any, new_text: str) -> None:
286+
new_text = new_text.lstrip()
287+
286288
if not self.view.controller.is_in_editor_mode():
287289
return
288290
# wait for any previously started search to finish to avoid
@@ -369,6 +371,8 @@ def __init__(self, topics_btn_list: List[Any], view: Any,
369371

370372
@asynch
371373
def update_topics(self, search_box: Any, new_text: str) -> None:
374+
new_text = new_text.lstrip()
375+
372376
if not self.view.controller.is_in_editor_mode():
373377
return
374378
# wait for any previously started search to finish to avoid
@@ -626,6 +630,9 @@ def update_user_list(self, search_box: Any=None,
626630
if not self.allow_update_user_list and new_text is None:
627631
return
628632

633+
if new_text is not None:
634+
new_text = new_text.lstrip()
635+
629636
# wait for any previously started search to finish to avoid
630637
# displaying wrong user list.
631638
with self.search_lock:

0 commit comments

Comments
 (0)