-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
feat: Use terminal title as the terminal tab title (#1298) #1302
feat: Use terminal title as the terminal tab title (#1298) #1302
Conversation
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
When the shell is zsh, the terminal title is the program command when an program is running, otherwise it’s “zsh”. Signed-off-by: Qian Qian "Cubik" <[email protected]>
6ccba6f
to
cb0297a
Compare
This is what we've got for now (with cb0297a): 2023-06-13.17-08-30.mp4There is a problem:
|
If worse comes to worse, as long as bash is titled bash, that is fine for now. Ideally it would work for both but I don't think we should get too hung up on this at this point in the project. We should really focus on our core experience. While our terminal is important, it is secondary to our editing experience. We do need to figure out a way for it to not be in the users history though. |
bash can probably work, it just needs a more complicated script, we are looking into it, i think it can be done soon |
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
Signed-off-by: Qian Qian "Cubik" <[email protected]>
@austincondiff bash part is also done 🎉 |
2023-06-13.19-10-49.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Beautiful! Great work @Cubik65536!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look good! Thanks for putting in the work on this one and for being patient!
@allcontributors add @Cubik65536 for code |
I've put up a pull request to add @Cubik65536! 🎉 |
Signed-off-by: Qian Qian "Cubik" [email protected]
Description
Currently, on the main branch, when we create a terminal in debug area, the title of a terminal tab is always "terminal". This PR changes the current behavior so that when there's no program running, the title is the name of the running shell (e.g. zsh, bash, etc.), and when a program is running, the title changes to the name of running program. That is, unless the user customized the title, then it will not be updated automatically.
Related Issues
Checklist
Use process name instead of the terminal title (at least for the moment) so every shell would work (as discussed in Discord).Screenshots
Video Showcase:
2023-06-07.14-28-26.mp4