vercel/hyper

Confirm closing

rastasheep opened this issue · 8 comments

It would be great to have some variation or both (in app or via plugins) of this iTerm features:

  • Confirm closing multiple sessions
  • Confirm "Quit iTerm2 (⌘+Q) command"

Both of those prevent accidental closing of app (and loosing work if you're not using some multiplexer).

That iTerm feature is the reason why I no longer use it. While it prevents accidental closing of a session, it also blocks things like restarts.

No installed program on the planet should block a system restart command, ever.

emmby commented

Hyperterm and iTerm should not block restarts, yes. However, it would be nice to have a (optional) warning before a non-restart quit/close-window. I write this having just now accidentally closed my window twice during long builds.

(This is a particular problem for people with multiple monitors, where it's easy to think one monitor has focus but in fact your other monitors does. First world problems.)

Just now I accidentally closed hyper window and that's it. Can't wait when this feature will be added.

@emmby - I agree completely. An optional setting would be valuable and as long as it doesn't block things like system restart, on by default would be perfectly fine.

I would widen scope a little of this request for a confirmation step to include accidental cmd+R, which I just did 🤦‍♂️

Is protection about hotkeys (quit app, quit pane or refresh app) enough?
Or should we warn if a user is closing app window with red button/cross?

More work about this will be tracked over #67