Skip to content

Conversation

v-zbsail
Copy link
Contributor

No description provided.

@ghost ghost requested review from vatsan-madhavan, rladuca, ryalanms and stevenbrix June 18, 2019 12:43
@ghost ghost added the PR metadata: Label to tag PRs, to facilitate with triage label Jun 18, 2019
@vatsan-madhavan vatsan-madhavan added the auto_merge bot-command label Jun 18, 2019
@ghost
Copy link

ghost commented Jun 18, 2019

Hello @vatsan-madhavan!

Because this pull request has the auto_merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost
Copy link

ghost commented Jun 18, 2019

@msftbot, require at least 1 sign off.

@ghost
Copy link

ghost commented Jun 18, 2019

@msftbot, require approvals from any one of the following: @vatsan-madhavan, @rladuca, @SamBent, @stevenbrix, @ryalanms

@vatsan-madhavan
Copy link
Member

@msftbot, require signoff from @ryalanms

@ghost ghost merged commit 9a10af9 into dotnet:master Jun 18, 2019
vatsan-madhavan added a commit that referenced this pull request Jun 18, 2019
@vatsan-madhavan
Copy link
Member

@ryalanms , msftbot did something strange here and merged the PR before the rules were satisfied. Would you mind taking a look to ensure all is well with this PR?

<trans-unit id="LocalizedControlTypeHyperlink">
<source>hyperlink</source>
<target state="translated">hipervínculo</target>
<target state="translated">hyperlink</target>
Copy link
Member

Choose a reason for hiding this comment

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

LocalizedControlType* strings are used to register control types for UI Automation. These strings should be localized.

<trans-unit id="LocalizedControlTypeHyperlink">
<source>hyperlink</source>
<target state="translated">lien hypertexte</target>
<target state="translated">hyperlink</target>
Copy link
Member

Choose a reason for hiding this comment

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

All LocalizedControlType* strings should remain localized for all languages. Please review LocalizedControlTypeTable as well.

<trans-unit id="LocalizedControlTypeTreeView">
<source>tree view</source>
<target state="translated">visualizzazione albero</target>
<target state="translated">tree view</target>
Copy link
Member

Choose a reason for hiding this comment

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

And here...

<trans-unit id="LocalizedControlTypeImage">
<source>image</source>
<target state="translated">イメージ</target>
<target state="translated">image</target>
Copy link
Member

Choose a reason for hiding this comment

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

And here...

<trans-unit id="LocalizedControlTypeHyperlink">
<source>hyperlink</source>
<target state="translated">超链接</target>
<target state="translated">hyperlink</target>
Copy link
Member

Choose a reason for hiding this comment

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

The three that I see are hyperlink, image, and table. These should remain localized. Thanks.

@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2022
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
auto_merge bot-command PR metadata: Label to tag PRs, to facilitate with triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants