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

Standardized Plugins Downloads and Center Icons #11494

Merged
merged 9 commits into from
Aug 8, 2022

Conversation

yiliang114
Copy link
Contributor

fix(vsx-registry): Standardized Plugins Downloads and Center Icons
Signed-off-by: yiliang114 [email protected]

What it does

  1. Standardized plugins downloads.
  2. Downloads icon and reviews icon are not center a little, and it center icons of VSXExtensionsWidget.

How to test

  1. Start application and turn Extensions activity bar.
  2. Search something such as chinese.

image

Review checklist

Reminder for reviewers

In VS Code, the downloads of extension is formatted with en-US and it endWiths k no matter what language is set. And the reviews number is always kept as an Int.

image

const downloadCompactFormatter = new Intl.NumberFormat(undefined, { notation: 'compact', compactDisplay: 'short' } as any);

But if theia uses undefined to new Intl.NumberFormat, it will cause language inconsistency when my browser default language is different from theia's default language.

image

After modification:
image

@yiliang114
Copy link
Contributor Author

Why sometimes CI runs failed after long time? And if i try to rerun them which have ended with some errors, they are okey again.

@msujew msujew added vsx-registry Issues related to Open VSX Registry Integration localization issues related to localization/internalization/nls labels Aug 2, 2022
@msujew
Copy link
Member

msujew commented Aug 2, 2022

@yiliang114 We have some flaky tests in there which sometimes (randomly) like to fail, usually related to file access which just times out. See #10348.

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, the changes look good to me 👍

1. Update averageRating count logic like VS Code
2. Add averageRating stars title in extension editor

Signed-off-by: yiliang114 <[email protected]>
@yiliang114
Copy link
Contributor Author

image

Referring to the format of vscode's Chinese translation, I added a Chinese translation to theia. Now its performance in Chinese and English is respectively

"averageRating": "平均评分:{0} 分(共 5 分)"

image
image

packages/core/i18n/nls.json Outdated Show resolved Hide resolved
packages/vsx-registry/src/browser/vsx-extension.tsx Outdated Show resolved Hide resolved
@yiliang114 yiliang114 requested a review from msujew August 4, 2022 08:04
@msujew
Copy link
Member

msujew commented Aug 4, 2022

Hm, I just noticed that the translation workflow does not run on Forks, see #11525. I'll hope I can address this quickly, since merging it as it is would lead to problems later.

Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look good to me 👍 The new localizations will be added to the framework before the release as they do not work from a fork and we changed our approach.

@yiliang114
Copy link
Contributor Author

The changes look good to me 👍 The new localizations will be added to the framework before the release as they do not work from a fork and we changed our approach.

Thank you both @msujew @vince-fugnitto

@vince-fugnitto vince-fugnitto merged commit bdd3261 into eclipse-theia:master Aug 8, 2022
@vince-fugnitto vince-fugnitto added this to the 1.29.0 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
localization issues related to localization/internalization/nls vsx-registry Issues related to Open VSX Registry Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants