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

Recommend language pack extensions based on locale #44012

Closed
ramya-rao-a opened this issue Feb 20, 2018 · 9 comments
Closed

Recommend language pack extensions based on locale #44012

ramya-rao-a opened this issue Feb 20, 2018 · 9 comments
Assignees
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@ramya-rao-a
Copy link
Contributor

Now that we are pulling language support out from the default product and into extensions, we should recommend appropriate language pack extensions based on the users locale

@ramya-rao-a ramya-rao-a added the feature-request Request for new features or functionality label Feb 20, 2018
@ramya-rao-a ramya-rao-a added this to the February 2018 milestone Feb 20, 2018
@ramya-rao-a ramya-rao-a self-assigned this Feb 20, 2018
@ramya-rao-a ramya-rao-a added the verification-needed Verification of issue is requested label Feb 27, 2018
@ramya-rao-a
Copy link
Contributor Author

@dbaeumer @sandy081 Are we pulling out the bundled translations out of the product for 1.21? If not, then this recommendation feature should not go out as well.

@ramya-rao-a ramya-rao-a reopened this Feb 28, 2018
@ramya-rao-a ramya-rao-a removed the verification-needed Verification of issue is requested label Feb 28, 2018
@dbaeumer
Copy link
Member

@ramya-rao-a no we don't. However for this release we should recommend the extensions that are in the market place. In VS Code I do white list right now the bundled languages so that they get special treatment :-). They are here: https://github.com/Microsoft/vscode/blob/master/src\main.js#L283

@ramya-rao-a
Copy link
Contributor Author

@dbaeumer I have added a check so that we recommend language packs only for the non core languages.

FYI https://github.com/Microsoft/vscode/blob/9a940305cb69991aa3af112db473ba324b6978c5/src/vs/base/common/platform.ts#L106 is the language I am using to recommend the language packs

@ramya-rao-a ramya-rao-a added the verification-needed Verification of issue is requested label Feb 28, 2018
@ramya-rao-a
Copy link
Contributor Author

@danyeh Can you re-publish the 4 language packs at https://github.com/Microsoft/vscode-loc after updating your local vsce tool to the latest version? This will add correct tags to these extensions which will help in search.

We cannot verify the recommendation feature without these tags

@sandy081 sandy081 added the verified Verification succeeded label Mar 1, 2018
@sandy081
Copy link
Member

sandy081 commented Mar 1, 2018

Verified from dev.

@ramya-rao-a
Copy link
Contributor Author

@sandy081 How did you verify this? The tags aren't present in the extensions in the marketplace yet.

@sandy081
Copy link
Member

sandy081 commented Mar 1, 2018

@ramya-rao-a Published a dummy extension contributing localizations and verified that this extensin is recommended by running from sources. (Have to comment a check in the source though)

@ramya-rao-a
Copy link
Contributor Author

Can you point me to the dummy extension you published?

@sandy081
Copy link
Member

sandy081 commented Mar 2, 2018

I unpublished it after testing. It's simple. Add localizations contribution without any translation files and published. Made sure that extension was recommended (in code, hardcode the language to languageId you mentioned in the extension)

@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 14, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

3 participants