Skip to content

Conversation

IndigoShock
Copy link
Contributor

Summary

The goal is to make this page more concise and readable

  • Deleted an extraneous T in the third Remarks section.
  • Deleted the Remarks section at the top because under each example has their own Remarks section.

Fixes #Issue_Number (if available)

@rpetrusha rpetrusha added this to the September 2019 milestone Sep 11, 2019
@rpetrusha rpetrusha added the ✨ 1st-time dotnet-api-docs contributor! Indicates PRs from new contributors to the dotnet-api-docs repository label Sep 11, 2019
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

Thank you, @IndigoShock, for contributing to the dotnet/dotnet-api-docs repo and elmininating duplication on this page. I thought, though, that a better option is to leave the paragraph in the Remarks section at the top of the page and remove it for the individual method overloads.

@rpetrusha
Copy link

Thanks both for contributing to making the documentation more concise, @IndigoShock. I'll merge your PR now.

@rpetrusha rpetrusha merged commit e7522f1 into dotnet:master Sep 11, 2019
@IndigoShock IndigoShock deleted the updatingIsControlCharPage branch September 11, 2019 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ 1st-time dotnet-api-docs contributor! Indicates PRs from new contributors to the dotnet-api-docs repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants