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

Language pack recommendation - Locale unset #167540

Closed
lszomoru opened this issue Nov 29, 2022 · 3 comments · Fixed by #170302
Closed

Language pack recommendation - Locale unset #167540

lszomoru opened this issue Nov 29, 2022 · 3 comments · Fixed by #170302
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug confirmed Issue has been confirmed by VS Code Team member insiders-released Patch has been released in VS Code Insiders l10n-platform Localization platform issues (not wrong translations) verified Verification succeeded
Milestone

Comments

@lszomoru
Copy link
Member

lszomoru commented Nov 29, 2022

Testing #167105

I went through the "Locale unset" steps on an M1 Mac and here is what I have observed:

  • The scenario steps imply that after setting the display language, and restarting the machine VS Code would start up in French and no notification would be shown. In my case, VS Code started in English and a notification was shown to install the French Language Pack. After installing and restarting VS Code was in French.
  • The locale key was not present in the argv.json file.

It's interesting that after, I have uninstalled the French Language Pack and VS Code is still in French 🤷🏻‍♂️
I was also able to reproduce the same when running through "Scenario4". There is definitely something on macOS where the language on VS Code is not being set, and a notification is being shown to install the language pack for the locale that is specified in the argv.json file

@lramos15
Copy link
Member

I did not see the locale get set in argv.json on linux either.

@meganrogge
Copy link
Contributor

no locale set on windows either

@rzhao271 rzhao271 added bug Issue identified by VS Code Team member as probable bug confirmed Issue has been confirmed by VS Code Team member labels Nov 29, 2022
@rzhao271
Copy link
Contributor

rzhao271 commented Nov 29, 2022

I think I confused the behaviour of "Configure Display Language" with the behaviour of installing a language extension and restarting; the former changes the value in the file. I believe that the language should still be configured in the argv.json file after installing the extension pack if it is installed by clicking on the "Install and restart" button.

@rzhao271 rzhao271 added this to the December 2022 milestone Dec 1, 2022
@TylerLeonhardt TylerLeonhardt added the l10n-platform Localization platform issues (not wrong translations) label Dec 5, 2022
TylerLeonhardt added a commit that referenced this issue Dec 30, 2022
@vscodenpa vscodenpa added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Dec 30, 2022
@sbatten sbatten added the verified Verification succeeded label Jan 27, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Feb 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Issue identified by VS Code Team member as probable bug confirmed Issue has been confirmed by VS Code Team member insiders-released Patch has been released in VS Code Insiders l10n-platform Localization platform issues (not wrong translations) verified Verification succeeded
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants