Skip to content

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

Merged
mdo merged 1 commit intoprimer:masterfrom
nickserv:patch-1
Apr 6, 2015
Merged

Remove an extra "the" from the primary button docs#97
mdo merged 1 commit intoprimer:masterfrom
nickserv:patch-1

Conversation

@nickserv
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
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
Copy Markdown
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