Skip to content

Adding a .text-center class to utilities#133

Merged
mdo merged 2 commits into
masterfrom
text-center
Jun 24, 2015
Merged

Adding a .text-center class to utilities#133
mdo merged 2 commits into
masterfrom
text-center

Conversation

@jonrohan

Copy link
Copy Markdown
Member

I found myself wanting to use .text-center and realized we didn't have anything like that. This pull adds one to the _utility.scss.

As an aside I got a warning from npm now that we didn't have a license in our package.json so I added it.

@primer/owners

@mdo mdo added this to the v2.3.0 milestone Jun 24, 2015
@mdo

mdo commented Jun 24, 2015

Copy link
Copy Markdown
Contributor

Nice, sounds like a good addition to me. There are a few things I think we can ship as a 2.3.0 release this week once we get this in.

@jasonlong

Copy link
Copy Markdown
Contributor

Cool. I thought we had something like this a couple times and then realized we didn't. 👍 from me.

@cobyism

cobyism commented Jun 24, 2015

Copy link
Copy Markdown

👍⚡

mdo added a commit that referenced this pull request Jun 24, 2015
Adding a .text-center class to utilities
@mdo mdo merged commit 0ad183b into master Jun 24, 2015
@mdo mdo deleted the text-center branch June 24, 2015 18:54
@mdo

mdo commented Jun 24, 2015

Copy link
Copy Markdown
Contributor

Good stuff, thanks dude :D.

tv104 pushed a commit to tv104/primer that referenced this pull request Aug 12, 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