Skip to content

Fixed spelling mistakes and comma punctuation errors#1530

Merged
gregvanl merged 1 commit intomicrosoft:masterfrom
wyckster:fix-spelling-mistakes
Mar 28, 2018
Merged

Fixed spelling mistakes and comma punctuation errors#1530
gregvanl merged 1 commit intomicrosoft:masterfrom
wyckster:fix-spelling-mistakes

Conversation

@wyckster
Copy link
Copy Markdown

I fixed several spelling mistakes and punctuation errors.

I know the use of a comma after a short introductory phrase is considered optional for phrases of four words or less, but Microsoft's grammar checker prefers the comma to be there, so I made the changes. I hope that's not contentious.

@msftclas
Copy link
Copy Markdown

msftclas commented Mar 28, 2018

CLA assistant check
All CLA requirements met.

- Switch to the color theme that you want to modify.
- Open the [settings](/docs/getstarted/settings.md) and make changes to view and editor colors using the `workbench.colorCustomizations`. Changes are applied live to your VS Code instance and no refreshing or reloading is necessary.
- A complete list of all themable colors can be found in the [color reference](https://code.visualstudio.com/docs/getstarted/theme-color-reference).
- A complete list of all themeable colors can be found in the [color reference](https://code.visualstudio.com/docs/getstarted/theme-color-reference).
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a tricky one. We've gone with themable in our APIs so we'll stick with themable in our docs as well. I'll revert after I merge the PR.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Ah! Understood. Thanks :)

@gregvanl gregvanl merged commit 8022cca into microsoft:master Mar 28, 2018
@gregvanl
Copy link
Copy Markdown

Thanks!

@Loridawn99
Copy link
Copy Markdown

Love you and miss you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants