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 files via upload #17037

Closed
wants to merge 1 commit into from
Closed

Add files via upload #17037

wants to merge 1 commit into from

Conversation

Ismith507
Copy link

@Ismith507 Ismith507 commented Apr 10, 2024

Summary of the Pull Request

Edited Resources.resw to reflect the style guidlines

References and Relevant Issues

Issue #16846

Detailed Description of the Pull Request / Additional comments

Edited Resources.resw to reflect the desired style guidelines. Specifically sentence style capitalization and use of elipses.

Validation Steps Performed

PR Checklist

@@ -187,6 +187,9 @@
<data name="Quit" xml:space="preserve">
<value>Quit</value>
</data>
<data name="CloseWindowWarningTitle" xml:space="preserve">
Copy link
Member

Choose a reason for hiding this comment

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

This resource seems to not be used anywhere?

@@ -215,10 +218,10 @@
<value>Web search</value>
</data>
<data name="TabColorChoose" xml:space="preserve">
<value>Change tab color</value>
<value>Color...</value>
Copy link
Member

Choose a reason for hiding this comment

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

isn't this literally the opposite of what was done in #16886

</data>
<data name="TabColorCustomButton.Content" xml:space="preserve">
<value>Custom</value>
<value>Custom...</value>
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 too)

@@ -835,7 +835,7 @@
<value>Close this tab</value>
</data>
<data name="NewTabMenuFolderEmpty" xml:space="preserve">
<value>Empty</value>
<value>Empty...</value>
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 too)

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Apr 23, 2024
@DHowett
Copy link
Member

DHowett commented Apr 23, 2024

"the opposite of ..."

I think what happened here is that lsmith took a snapshot of the file as it was when they started working, and re-uploaded it via the GitHub web UI after they made their modifications.

If that's the case - there are some changes that need to be integrated from #16886!

@Ismith507
Copy link
Author

Hi, I can integrate changes from #16886 and reupload! After that, is there anything else to be done before I can merge? or does #16886 fix the problem and is this PR now redundant?

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Apr 24, 2024
@zadjii-msft
Copy link
Member

Okay we looked over it all again - looks like #16886 took care of things here. Thanks anyways!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs-Attention The core contributors need to come back around and look at this ASAP.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UI style errors: Menu items capitalization and … (ellipses) mark misuse
3 participants