Clear multi-select filter when option is selected

AlecAivazis / survey

A golang library for building interactive and accessible prompts with full support for windows and posix terminals.

MIT License

4.09k stars 355 forks source link

Clear multi-select filter when option is selected #158

Closed wilhelmeek closed 5 years ago

wilhelmeek commented 5 years ago

This removes the need to manually clear the filter when you want to change the input after selecting an option in multi-select.

AlecAivazis commented 5 years ago

Hey @wilhelmeek - thanks for implementing this!

I know its a small change, but would you mind adding a test to verify this behavior?

wilhelmeek commented 5 years ago

@AlecAivazis for sure!

wilhelmeek commented 5 years ago

@AlecAivazis I've added a prompt test that tests this behaviour. Not sure if this is would be comprehensive enough to assert this behaviour, so please let me know if you have alternative suggestions 😄

AlecAivazis commented 5 years ago

Thanks @wilhelmeek!

I think its enough because if it didn't clear the filter, Tuesday would have stayed highlighted 👍