-
Notifications
You must be signed in to change notification settings - Fork 322
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
Bug: Losing padding style with advanced: true
#452
Labels
Comments
That's interesting indeed. Fix is coming! |
Thanks @jakubpawlowicz! Haha yeah, I’m curious what the fix will look like. 😜 |
It's fine in 2.2 though. |
jakubpawlowicz
added a commit
that referenced
this issue
Feb 4, 2015
It's a regression from 2.2 as we added a smarter merging but it does not always correctly compare arrays (bloody JavaScript ;-).
@adamschwartz Fixed in 3.0.9 - here's the fix 612219e#diff-99a30d068c42b4f98cbbb103231cc2eaL225 kinda backported from 2.2. |
Awesome. Thanks so much! 👍 |
Sure thing. Thanks for reporting! 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In
v3.0.8
, the this CSS:minifies to:
Essentially the rule
padding-bottom: 1.25em;
is omitted.The text was updated successfully, but these errors were encountered: