-
-
Notifications
You must be signed in to change notification settings - Fork 319
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
mergeLonghand is NOT SAFE! here is a case #461
Comments
@fjc0k what version |
@evilebottnawi 3.10.0 |
@fjc0k can you try latest version |
|
@fjc0k btw, what wrong with merge? |
I think what @fjc0k means is that the merge result should be
As is the merge result overwrites the whole |
@evilebottnawi sorry, now i can’t test it. The worng is as @prettyv said. |
@fjc0k what result expected? |
@evilebottnawi border: 0 solid rgba(0, 0, 0, 0.2); border-width: 1px; Sent with GitHawk |
@fjc0k Are you sure that border: 0 solid rgba(0, 0, 0, 0.2);
border-top-width: 1px;
border-right-width: 1px;
border-bottom-width: 1px;
border-left-width: 1px; must be border: 0 solid rgba(0, 0, 0, 0.2);
border-width: 1px; ? Because I tested it locally, and i got this border: 1px solid rgba(0, 0, 0, 0.2); |
@An-Tu Oh, I think border: 0 solid rgba(0, 0, 0, 0.2);
border-width: 1px; is equal to border: 1px solid rgba(0, 0, 0, 0.2); What I want to say is that the result is wrong, not the form. |
Already fixed in 4.0.0-rc.0. I think this issue might be closed. |
raw css (generated by stylus mixin):
minified css with
mergeLonghand = true
:The text was updated successfully, but these errors were encountered: