Skip to content

Add CSS Columns Specification [WIP] #541

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

Closed
wants to merge 10 commits into from
Closed

Add CSS Columns Specification [WIP] #541

wants to merge 10 commits into from

Conversation

codytooker
Copy link
Contributor

@codytooker codytooker commented Aug 26, 2018

This PR adds a handful of new utilities covering the css column spec. This is a work in progress and just a proposal on what utilities could be added to cover css columns.

Added Utilities For

  • Column Count
  • Column Fill
  • Column Span
  • Column Width
  • Column Rule Width
  • Column Rule Style
  • Column Rule Color

As of now these are all separated into their own generators but can easily be converted into one generator to clean it up.

Config Updates

There are some updates to the default config for column-count, column-width, column-rule-wdith, and column-rule-colors, however all of these except for column count could theoretically pull from other config options. The three column rule specifications are essentially the same as border so we could use those same settings. Column Width could pull from the width settings as well.

As Css columns is not the most commonly used system in css these generators could be set to false in the default config, but I do think they are useful for their purpose and can be useful in the Tailwind Core.

Progrees

  • Generators
  • Config
  • Docs
  • Testing / Fixtures

Topics of Discussion

  • Converting generators to one simple generator
  • Using already established border config settings for the three Column Rule classes
  • Default variants or false

@codytooker codytooker mentioned this pull request Aug 29, 2018
@codytooker
Copy link
Contributor Author

codytooker commented Aug 31, 2018

Updated all classes to use .col- instead of .column-

@adamwathan
Copy link
Member

Hey really sorry to not have addressed this at all until now but I think this would be better suited for a plugin right now. Really appreciate the contribution either way and sorry again for not acknowledging the PR sooner.

@adamwathan adamwathan closed this Mar 12, 2019
@codytooker
Copy link
Contributor Author

Definitely. After building 20 or so sites with tailwind I only used this once. So a plug-in is definitely a better fit.

@hacknug
Copy link
Contributor

hacknug commented Mar 13, 2019

I published a plugin while ago inspired by your comments on #540. Links just for reference, in case someone lands here and doesn't look into the related issues:

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.

3 participants