-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
support Pseudoterminal.onDidChangeName #11657
Conversation
Implement support for Pseudoterminal.onDidChangeName VS Code API by listening to this event on the plugin side and updating Theia's terminal widget's title accordingly.
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.
In addition to @paul-marechal's comments, I have some minor ones related to the formatting of the documentation. I confirmed that the functionality works as expected.
Co-authored-by: Paul Maréchal <paul.marechal@ericsson.com>
Co-authored-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
Co-authored-by: Vincent Fugnitto <vincent.fugnitto@ericsson.com>
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.
LGTM 👍 The changes look good, and work well with the provided plugin.
I can see that the initial terminal name is displayed, and updated properly.
What it does
Closes #11513
Adds support for the Pseudoterminal.onDidChangeName VS Code API.
How to test
Review checklist
Reminder for reviewers