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

Remove an extra "the" from the primary button docs #97

Merged
merged 1 commit into from
Apr 6, 2015

Conversation

nickserv
Copy link
Contributor

@nickserv nickserv commented Apr 5, 2015

It seems like an extra "the" slipped in with a recently merged pull request. I also removed a hyphen that was being used incorrectly.

@djch
Copy link

djch commented Apr 6, 2015

@nicolasmccurdy That's an em dash—not a hyphen—and I think that's valid usage of it?

@nickserv
Copy link
Contributor Author

nickserv commented Apr 6, 2015

Oh I'm sorry, I thought it was a hyphen with the diff's monospaced font. It totally is an em dash, and it's being used correctly. I'll fix this now.

@nickserv nickserv changed the title Grammar tweaks for the primary button docs Remove an extra "the" from the primary button docs Apr 6, 2015
mdo added a commit that referenced this pull request Apr 6, 2015
Remove an extra "the" from the primary button docs
@mdo mdo merged commit 7d296bd into primer:master Apr 6, 2015
@mdo mdo added this to the v2.1.0 milestone Apr 6, 2015
@mdo mdo added the docs label Apr 6, 2015
@nickserv nickserv deleted the patch-1 branch April 6, 2015 13:53
@jonrohan
Copy link
Member

jonrohan commented Apr 6, 2015

Thanks @nicolasmccurdy, my bad.

@mdo mdo mentioned this pull request Apr 15, 2015
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