fix: merge-rule before longhand in default preset #886
Conversation
Codecov Report
@@ Coverage Diff @@
## master #886 +/- ##
=======================================
Coverage 97.27% 97.27%
=======================================
Files 118 118
Lines 3452 3452
Branches 1036 1036
=======================================
Hits 3358 3358
Misses 86 86
Partials 8 8
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #886 +/- ##
=======================================
Coverage 97.26% 97.26%
=======================================
Files 119 119
Lines 3477 3477
Branches 1047 1047
=======================================
Hits 3382 3382
Misses 87 87
Partials 8 8
Continue to review full report at Codecov.
|
| .process(css, { from: undefined }) | ||
| .then((result) => { | ||
| expect(result.css).toBe( | ||
| '.bar,.foo{border-color:#cc1f1a}.bar{border-bottom:1px}' |
There was a problem hiding this comment.
This output according to the issue #701 (comment) is wrong, not sure why it is wrong in the issue as this is the safe merging in my opinion.
according to the issue, it should be
.foo {
border-color: #cc1f1a;
}
.bar {
border-bottom: 1px;
border-color: #cc1f1a;
}Though I will check this once again, @evilebottnawi thoughts ?
There was a problem hiding this comment.
yes, it is bug, need to fix
fixes #701
#701 (comment)