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

Prompt for language pack only in English #53130

Closed
chrmarti opened this issue Jun 27, 2018 · 9 comments
Closed

Prompt for language pack only in English #53130

chrmarti opened this issue Jun 27, 2018 · 9 comments
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug important Issue identified as high-priority L10N Translation issue l10n-platform Localization platform issues (not wrong translations) upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Milestone

Comments

@chrmarti
Copy link
Contributor

Testing #52960

  • VSCode Version: 1.25 build for testing
  • OS Version: High Sierra

The test item mentions it should also be in the OS language. That does not seem to be the case.

image

@chrmarti chrmarti added the L10N Translation issue label Jun 27, 2018
@sandy081 sandy081 assigned ramya-rao-a and unassigned sandy081 Jun 27, 2018
@ramya-rao-a
Copy link
Contributor

This is because the language pack for French doesn't have the required strings.
For that matter, almost none of the core language packs have the required strings.

@danyeh,
The required translations are in the built-in language support but not in the language pack extensions.

https://github.com/Microsoft/vscode/search?q=installAndRestartMessage&unscoped_q=installAndRestartMessage

https://github.com/Microsoft/vscode-loc/search?q=installAndRestartMessage&unscoped_q=installAndRestartMessage

Can you ensure that these strings get into the language packs?

@ramya-rao-a ramya-rao-a assigned danyeh and unassigned ramya-rao-a Jun 27, 2018
@ramya-rao-a ramya-rao-a added this to the June 2018 milestone Jun 27, 2018
@ramya-rao-a ramya-rao-a added the bug Issue identified by VS Code Team member as probable bug label Jun 27, 2018
@isidorn isidorn added the upstream Issue identified as 'upstream' component related (exists outside of VS Code) label Jun 29, 2018
@sandy081
Copy link
Member

sandy081 commented Jun 29, 2018

@danyeh Following language packs are not missing translations for those necessary strings which we show to user to install the language pack before installing it.

  • Necessary translations
showLanguagePackExtensions: localize('showLanguagePackExtensions', "Search language packs in the Marketplace to change the display language to {0}."),
	searchMarketplace: localize('searchMarketplace', "Search Marketplace"),
	installAndRestartMessage: localize('installAndRestartMessage', "Install language pack to change the display language to {0}."),
	installAndRestart: localize('installAndRestart', "Install and Restart")
  • de
"vs/platform/node/minimalTranslations": {
	"searchMarketplace": "Marketplace durchsuchen",
	"installAndRestart": "Installieren und neu starten"
}
  • es
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Buscar en Marketplace ",
    "installAndRestart": "Instalar y Reiniciar"
}
  • fr
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Rechercher dans le Marketplace",
    "installAndRestart": "Installer et Redémarrer"
}
  • it
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Cerca nel Marketplace",
    "installAndRestart": "Installa e riavvia"
}
  • ja
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Marketplace を検索",
    "installAndRestart": "インストールして再起動"
}
  • ko
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Marketplace 검색",
    "installAndRestart": "설치 및 다시 시작"
}
  • ru
"vs/platform/node/minimalTranslations": {
    "searchMarketplace": "Поиск в Marketplace",
    "installAndRestart": "Установить и перезапустить"
}

-zh-tw: No translations at all

Only zh-cn has translations for all strings.

I would request to fix the core language packs and make sure they have necessary translations before we release the stable without core languages.

@sandy081 sandy081 added the l10n-platform Localization platform issues (not wrong translations) label Jun 29, 2018
@sandy081 sandy081 assigned sandy081 and ramya-rao-a and unassigned danyeh Jun 29, 2018
@sandy081 sandy081 added the important Issue identified as high-priority label Jun 29, 2018
@ramya-rao-a
Copy link
Contributor

-zh-tw: No translations at all
Only zh-cn has translations for all strings.

@sandy081 When users choose zh-cn or zh-tw we fall back to zh-hans or zh-hant respectively. The pack for zh-hans has the required translations, zh-hant doesnt.

@danyeh
Copy link
Contributor

danyeh commented Jun 29, 2018

According to loc schedule, we will cover all strings by end of day July 3rd. And publish language pack afterward.

@sandy081
Copy link
Member

sandy081 commented Jul 2, 2018

When users choose zh-cn or zh-tw we fall back to zh-hans or zh-hant respectively. The pack for zh-hans has the required translations, zh-hant doesnt.

Fallback is only for extension name and locale id contributed by respective extensions still should be zh-cn and zh-tw.

@danyeh
Copy link
Contributor

danyeh commented Jul 2, 2018

As of today, we have zh-cn, fr, it, ja, ko language pack loc completed and published in Marketplace.

@sandy081
Copy link
Member

sandy081 commented Jul 3, 2018

Thanks @danyeh for the update.

  • zh-cn: This needed fix from our side
    image

  • fr
    image

  • it
    image

  • ja
    image

  • ko
    image

sandy081 added a commit that referenced this issue Jul 3, 2018
sandy081 added a commit that referenced this issue Jul 3, 2018
@danyeh
Copy link
Contributor

danyeh commented Jul 3, 2018

Remaining language pack loc completed and published in Marketplace - de, es, ru, zh-tw.

@sandy081
Copy link
Member

sandy081 commented Jul 4, 2018

Verified all core language packs showing recommendation prompt in their respective languages.

@sandy081 sandy081 closed this as completed Jul 4, 2018
@sandy081 sandy081 added the verified Verification succeeded label Jul 4, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Aug 18, 2018
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 important Issue identified as high-priority L10N Translation issue l10n-platform Localization platform issues (not wrong translations) upstream Issue identified as 'upstream' component related (exists outside of VS Code) verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants