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

[feature] add configurable behavior when session ends #8206

Merged
merged 7 commits into from
Apr 24, 2023

Conversation

Clem-Fern
Copy link
Contributor

Hi @Eugeny,

This PR adds different tab's behaviors on session ending.
As asked in #6617, #7751 and #8062, you can choose to keep the tab opened after you got disconnected or exited the shell. You can also choose to automatically reconnect the session or close the tab.

image

I didn't add the ability to configure this feature independently on each profile, but i can definitely take a look if you think it could be useful.

Feel free to ask if any changes are required.

tabby-ssh/src/components/sshTab.component.ts Show resolved Hide resolved
tabby-serial/src/components/serialTab.component.ts Outdated Show resolved Hide resolved
tabby-terminal/src/config.ts Outdated Show resolved Hide resolved
@Clem-Fern Clem-Fern marked this pull request as draft April 15, 2023 09:02
@Clem-Fern Clem-Fern changed the title [feature] add configurable behavior when session ends [wip][feature] add configurable behavior when session ends Apr 15, 2023
@Clem-Fern Clem-Fern marked this pull request as ready for review April 23, 2023 08:51
@Clem-Fern
Copy link
Contributor Author

Hi @Eugeny :)

As discuss, I've made the changes requested. You can now select 4 different options to choose how the tab should behave when a session ends.

image

I voluntarily set the default mode to auto as it is most similar to the actual behavior.

Feel free to ask if any changes are required.

@Clem-Fern Clem-Fern changed the title [wip][feature] add configurable behavior when session ends [feature] add configurable behavior when session ends Apr 23, 2023
@Eugeny
Copy link
Owner

Eugeny commented Apr 24, 2023

LGTM, thank you! I'll keep testing it the next week.

@Eugeny Eugeny merged commit ff66f05 into Eugeny:master Apr 24, 2023
@Clem-Fern Clem-Fern deleted the feat#7751 branch May 11, 2023 19:15
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

Successfully merging this pull request may close these issues.

2 participants