-
Notifications
You must be signed in to change notification settings - Fork 46
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-aspect-ratio #42
Comments
@seaneking would you accept a PR with this? |
whoops sorry missed this. Not sure if aspect ratio us a good fit for rucksack or not, it's quite heavy handed it what it does. As a developer looking through code with that declaration in it, I wouldn't expect the children of the element to be absolutely positioned etc etc. More than happy to get feedback from the community on this one though, if there's demand for it then would definitely consider it. |
It's a very standard (and crucial imho) technique, but obviously it's your call. :) |
The padding hack is, but all the other assumptions postcss-aspect ratio make aren't (abspos children etc). That said haven't looked at it for a while. |
There's no other way to do content inside it without it. |
Would love to see this functionality added, its a nice shortcut for a common css code pattern 👬 |
+1 for aspect ratios! I find myself needing to do that more often that I would have thought. |
Okay, sounds like I'm the minority on this one. Would accept a PR. While we're all here - what do we think of #47? FWIW postcss-alias is pretty buggy, and the bugs are pretty fundamental (regexing aliases out of both property names and values). |
sweet. I'll PR this |
Excited that this will be added - another vote for it! |
Seems perfect for Rucksack.. something everyone has to have in their responsive toolkit.
https://github.com/arccoza/postcss-aspect-ratio
Happy to attempt a PR if you want it.
The text was updated successfully, but these errors were encountered: