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

[docs] Some grammar/text edits #6976

Merged
merged 1 commit into from
May 27, 2017
Merged

[docs] Some grammar/text edits #6976

merged 1 commit into from
May 27, 2017

Conversation

sghall
Copy link
Contributor

@sghall sghall commented May 27, 2017

Just reading through the new docs and firing off some suggested edits.

  • PR has tests / docs demo, and is linted.
  • Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
  • Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

Just reading through the new docs and firing off some suggested edits.
@oliviertassinari oliviertassinari added docs Improvements or additions to the documentation PR: Review Accepted labels May 27, 2017
@oliviertassinari
Copy link
Member

@sghall Thank you!

@oliviertassinari oliviertassinari merged commit 8252151 into mui:next May 27, 2017
Themes let you apply a consistent tone to an app.
It allows you to **customize all the design aspects** in order to meet the needs of UI diversity from business and brand.
Themes let you apply a consistent tone to your app.
It allows you to **customize all design aspects** of your project in order to meet the specific needs of your business or brand.
Copy link
Member

Choose a reason for hiding this comment

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

"all the design aspects" was correct.

@oliviertassinari oliviertassinari changed the title Some grammar/text edits [docs] Some grammar/text edits Jun 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants