Skip to content

Conversation

@adamwathan
Copy link
Member

This PR enables the following variants by default:

  • Focus variants for background colors
  • Focus variants for border colors
  • Focus variants for font weight
  • Hover variants for box shadows
  • Focus variants for text colors
  • Focus variants for text style (underline, capitalization, etc.)

It adds significant size to the CDN builds but it makes the default build much more useful out of the box. We have thorough documentation on managing build size so I think it's silly to cripple the default build just to make the framework appear smaller to people who don't spend the time to learn about how it actually works.

Copy link
Member

@reinink reinink left a comment

Choose a reason for hiding this comment

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

go

@adamwathan adamwathan merged commit 9339839 into master Jun 20, 2018
@MichaelDeBoey
Copy link
Contributor

@adamwathan Don't forget to change the variants in the docs either. (Do I make a PR for it?)
Maybe we can get the variants from the config in some way, so this isn't any problem anymore in the future?

@adamwathan
Copy link
Member Author

@MichaelDeBoey Yeah definitely need to do that, would welcome a PR if you're around to do it today, if not I'll get to it myself at some point before the end of the day 👍

@MichaelDeBoey
Copy link
Contributor

@adamwathan PR is on it's way 😉

@MichaelDeBoey
Copy link
Contributor

MichaelDeBoey commented Jun 21, 2018

@adamwathan You can check tailwindlabs/tailwindcss.com#25 for the PR 😉

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