Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confirm quit #355

Open
Omnikron13 opened this issue Apr 11, 2024 · 0 comments
Open

Confirm quit #355

Omnikron13 opened this issue Apr 11, 2024 · 0 comments

Comments

@Omnikron13
Copy link

Accidentally hitting esc/q and losing the flow of what you are doing isn't the end of the world, but atm it seems a little too easy to accidentally do.

As it stands those keybinds insta-quit the dashboard, even in circumstances where instincts from other UIs & general expectations suggest it shouldn't.

Most notably, when you have hit ? and have the 'help' (cheatsheet?) open, it is quite easy to absent mindedly expect at least esc to just close that, not exit the entire program.

I'd propose hitting those keys when you most recently opened up a kinda sub-window like that, it should close those in reverse order before attempting to quit the whole dashboard.

I think perhaps a 'really quit?' prompt would be a nice QoL improvement. Imo having quick and easy options to confirm the quit (e.g. hitting esc or q again, hitting y, hitting return, etc.) should mean the nag is outweighed by avoiding the flow-breaking drop back to the terminal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant