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

add postcss mixins #123

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add postcss mixins #123

wants to merge 1 commit into from

Conversation

lux-v
Copy link
Contributor

@lux-v lux-v commented Nov 18, 2024

TODO: figure out how to import mixins globally

@lux-v lux-v requested a review from dChiamp November 18, 2024 22:12
@drewbaker
Copy link
Member

@dChiamp how do you feel about this? I tend to think mixins are very hard to debug. I’d rather use a utility class, like “has-underline” personally.

@dChiamp
Copy link
Contributor

dChiamp commented Nov 18, 2024

@drewbaker Occasionally i think they can be useful, but yeah i guess it could just be a utility class in this case.

The bigger question is do we want to add some of the post-css modules to the base repo, like iterators?
I'm liking post-css but adding a bunch of modules to each project is annoying. Nice job getting this one working tho @lux-v

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