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

Extract display language patch and submit PR upstream #4703

Closed
jsjoeio opened this issue Jan 7, 2022 · 1 comment
Closed

Extract display language patch and submit PR upstream #4703

jsjoeio opened this issue Jan 7, 2022 · 1 comment
Assignees
Labels
feature New user visible feature
Milestone

Comments

@jsjoeio
Copy link
Contributor

jsjoeio commented Jan 7, 2022

@code-asher recently patched #4598 in coder/vscode#34 (which we've had for a while but lost in a vscode upgrade).

We noticed this issue exists upstream in Codespaces, VS Code Web and have discussed extracting our patch and submitting a PR upstream.

The extraction will require some tweaking, but hoping it's not too difficult.

Related

@jsjoeio jsjoeio added the feature New user visible feature label Jan 7, 2022
@jsjoeio jsjoeio added this to the 4.0.2 milestone Jan 7, 2022
@jsjoeio jsjoeio self-assigned this Jan 18, 2022
@code-asher code-asher modified the milestones: 4.0.2, 4.0.3 Jan 27, 2022
@jsjoeio jsjoeio modified the milestones: 4.0.3, On Deck Feb 1, 2022
@jsjoeio jsjoeio modified the milestones: On Deck, March 2022 Mar 3, 2022
@jsjoeio jsjoeio modified the milestones: March 2022, April 2022 Mar 22, 2022
@jsjoeio jsjoeio modified the milestones: April 2022, May 2022 Apr 26, 2022
@jsjoeio
Copy link
Contributor Author

jsjoeio commented Jun 15, 2022

They ended up going with a different approach. See microsoft/vscode#146012 (comment)

@jsjoeio jsjoeio closed this as not planned Won't fix, can't repro, duplicate, stale Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New user visible feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants