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

Add tips for debug views #205861

Merged
merged 3 commits into from
Feb 21, 2024
Merged

Conversation

yiliang114
Copy link
Contributor

@yiliang114 yiliang114 commented Feb 21, 2024

The wrong translation

image

However, I know that the repair of the copy needs to be vscode-loc in the warehouse, but a hint has been added here.

Such as
https://github.com/microsoft/vscode-loc/blob/b0beccc0d53c8f2353ac6d469cd5f68d2e0806d3/i18n/vscode-language-pack-zh-hans/translations/main.i18n.json#L6296

@@ -296,7 +296,7 @@ viewsRegistry.registerViewWelcomeContent(EmptyView.ID, {
});

viewsRegistry.registerViewWelcomeContent(EmptyView.ID, {
content: localize({ key: 'noFolderHelpWeb', comment: ['Please do not translate the word "commmand", it is part of our internal syntax which must not change'] },
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Could many comments here be deleted in fact? Because there is no command string in the actual copy.

Copy link
Member

Choose a reason for hiding this comment

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

It's possible, but I don't want to do that for another feature area in this PR

@@ -296,7 +296,7 @@ viewsRegistry.registerViewWelcomeContent(EmptyView.ID, {
});

viewsRegistry.registerViewWelcomeContent(EmptyView.ID, {
content: localize({ key: 'noFolderHelpWeb', comment: ['Please do not translate the word "commmand", it is part of our internal syntax which must not change'] },
Copy link
Member

Choose a reason for hiding this comment

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

It's possible, but I don't want to do that for another feature area in this PR

@roblourens roblourens added this to the February 2024 milestone Feb 21, 2024
@roblourens roblourens merged commit e7a3ef8 into microsoft:main Feb 21, 2024
6 checks passed
@microsoft microsoft locked and limited conversation to collaborators Jun 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants