seba
/
clintermission
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Return with "no success" when no options are given 

If the user gave us no options then we used to just raise a ValueError
(due to initial_pos being out of range). In most - if not all - cases
it would be more favorable to act the same way as if the selection has
been aborted, so that's what we're doing now.
This commit is contained in:
Sebastian Lohff 2022-10-26 12:26:36 +02:00
parent 1a309d28c0
commit 33aa41ac6e
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
clintermission/climenu.py
View File

@ -272,6 +272,10 @@ class CliMenu:
event.app.exit() event.app.exit()
def _run(self): def _run(self):
if self._item_num == 0:
self._success = False
return
self._preflight() self._preflight()
class MenuColorizer(Processor): class MenuColorizer(Processor):
@ -284,6 +288,7 @@ class CliMenu:
@self._kb.add('q', filter=~is_searching) @self._kb.add('q', filter=~is_searching)
@self._kb.add('c-c') @self._kb.add('c-c')
def quit(event): def quit(event):
self._success = False
event.app.exit() event.app.exit()
@self._kb.add('down', filter=~is_searching) @self._kb.add('down', filter=~is_searching)