cutie icon indicating copy to clipboard operation
cutie copied to clipboard

Unresponsivness when using select_multiple with minimal_count and hide_confirm

Open provinzkraut opened this issue 6 years ago • 3 comments

Using select_multiple with minimal_count and hide_confirm=True leads to unresponsivness.

When trying to confirm, the user isn't prompted with a feedback and the state just doesn't change. Keyboard interrupt via CTRL + C doesn't work either, the CLI is just unresponsive until the right amount of options are selected

Code to repoduce the issue: cutie.select_multiple(["foo", "bar"], minimal_count=1, hide_confirm=True)

provinzkraut avatar Nov 16 '18 21:11 provinzkraut

I just noticed that CTRL + C doesn't work at all when using minimal_count while the requirements are not met

provinzkraut avatar Nov 16 '18 21:11 provinzkraut

I just created a new branch https://github.com/Kamik423/cutie/tree/consistency. The first commit in it, https://github.com/Kamik423/cutie/commit/8e89baf54498cc83821d3a9a0b9147279d745046 (Default key list), should take care of at least CTRL + C in this case.

Kamik423 avatar Nov 16 '18 21:11 Kamik423

Is this fixed by https://github.com/Kamik423/cutie/commit/24659a95691a2dc05630aa36aa8b59f49e54d977?

Kamik423 avatar Nov 19 '18 17:11 Kamik423