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

fix for #22325: Support title from xterm.js #22684

Closed
wants to merge 1 commit into from
Closed

fix for #22325: Support title from xterm.js #22684

wants to merge 1 commit into from

Conversation

limorf
Copy link

@limorf limorf commented Mar 15, 2017

This small commit should fix #22325.

#goodnessSquad

@mention-bot
Copy link

@limorf, thanks for your PR! By analyzing the history of the files in this pull request, we identified @Tyriar and @dbaeumer to be potential reviewers.

@msftclas
Copy link

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@Tyriar
Copy link
Member

Tyriar commented Mar 15, 2017

@limorf someone else submitted a very similar PR here #22454 but I believe there are issues with this approach. See my comment here #22454 (comment)

@Tyriar Tyriar self-assigned this Mar 15, 2017
@Tyriar Tyriar self-requested a review March 15, 2017 18:47
@Tyriar
Copy link
Member

Tyriar commented Apr 3, 2017

Closing as it still needs follow up work

@Tyriar Tyriar closed this Apr 3, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support title from xterm.js
4 participants