[Feature Request] A confirmation dialect window show up before closing the current tab. #2629

Closed
opened 2026-01-30 23:00:33 +00:00 by claunia · 5 comments
Owner

Originally created by @were on GitHub (Jul 8, 2019).

Summary of the new feature/enhancement

Is it possible, to implement something like this:
When there is an active process in the current tab, a confirmation dialect window may show up before actually closing.

When I was writing vim, I need to use C-W to switch among vim tabs.
C-W is also the hotkey to close the current tab.
When the first two times I ran into this problem, I suppose it was some crashing thing.
Finally, I realized it was a hotkey conflict.

Proposed technical implementation details (optional)

I did not dive into the code base of this project yet, but I hope it is not too hard to implement.

Originally created by @were on GitHub (Jul 8, 2019). # Summary of the new feature/enhancement Is it possible, to implement something like this: When there is an active process in the current tab, a confirmation dialect window may show up before actually closing. When I was writing vim, I need to use `C-W` to switch among vim tabs. `C-W` is also the hotkey to close the current tab. When the first two times I ran into this problem, I suppose it was some crashing thing. Finally, I realized it was a hotkey conflict. # Proposed technical implementation details (optional) I did not dive into the code base of this project yet, but I hope it is not too hard to implement.
Author
Owner

@zadjii-msft commented on GitHub (Jul 8, 2019):

This seems like a good feature request. I think we've mentioned it somewhere else in another thread, but this is a good setting. Something like:

"promptBeforeClosing": bool

or similar, to control whether the dialog appears or not. Or something like

"suppressCloseDialog": bool

and display the dialog by default when we notice there's more than one child* process attached to the terminal

*: not including conhost of course

@zadjii-msft commented on GitHub (Jul 8, 2019): This seems like a good feature request. I think we've mentioned it somewhere else in another thread, but this is a good setting. Something like: ```json "promptBeforeClosing": bool ``` or similar, to control whether the dialog appears or not. Or something like ```json "suppressCloseDialog": bool ``` and display the dialog by default when we notice there's more than one child\* process attached to the terminal \*: not including conhost of course
Author
Owner

@were commented on GitHub (Jul 9, 2019):

Is there any branch implementing this? If not, I can take it.

@were commented on GitHub (Jul 9, 2019): Is there any branch implementing this? If not, I can take it.
Author
Owner

@zadjii-msft commented on GitHub (Jul 9, 2019):

@were nope, not yet. Feel free to take a stab at it!

@zadjii-msft commented on GitHub (Jul 9, 2019): @were nope, not yet. Feel free to take a stab at it!
Author
Owner

@zadjii-msft commented on GitHub (Nov 6, 2019):

Hey so this was partially implemented in #2526, to resolve #1589.

There's also #2976, which tracks a follow-up to #2526 to connect more settings to the dialog. Since that issue seems to have a few more details, I'm going to close this one in favor of that one. Thanks!

/dup #1589
/dup #2976

@zadjii-msft commented on GitHub (Nov 6, 2019): Hey so this was partially implemented in #2526, to resolve #1589. There's also #2976, which tracks a follow-up to #2526 to connect more settings to the dialog. Since that issue seems to have a few more details, I'm going to close this one in favor of that one. Thanks! /dup #1589 /dup #2976
Author
Owner

@ghost commented on GitHub (Nov 6, 2019):

Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!

@ghost commented on GitHub (Nov 6, 2019): Hi! We've identified this issue as a duplicate of another one that already exists on this Issue Tracker. This specific instance is being closed in favor of tracking the concern over on the referenced thread. Thanks for your report!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#2629