Cssnano: mergeLonghand removes things it shouldn't remove

Created on 17 Jul 2018  路  8Comments  路  Source: cssnano/cssnano

mergeLonghand removes CSS in [email protected].

Input:

.b {
    border-top-color: rgba(85, 85, 85, 0.95);
    border-bottom: none;
}

Output:

.b {
  border-bottom: none;
}
bug
Source
picture hudochenkov
👍4

Most helpful comment

@vseguin I disabled this rule:

require('cssnano')({
    preset: [
        'default',
        {
            mergeLonghand: false,
        },
    ],
}),
picture hudochenkov on 17 Jul 2018
👍2

All 8 comments

@hudochenkov Do you have any workaround? Did you downgrade meanwhile?

vseguin picture vseguin on 17 Jul 2018

@vseguin I disabled this rule:

require('cssnano')({
    preset: [
        'default',
        {
            mergeLonghand: false,
        },
    ],
}),
hudochenkov picture hudochenkov on 17 Jul 2018
👍2

/cc @andyjansson

alexander-akait picture alexander-akait on 17 Jul 2018

@hudochenkov worked for me, thank you so much!

vseguin picture vseguin on 17 Jul 2018

Looking into this.

andyjansson picture andyjansson on 17 Jul 2018
👍1

Done in https://github.com/cssnano/cssnano/pull/555

alexander-akait picture alexander-akait on 18 Jul 2018

Fixed in [email protected]

alexander-akait picture alexander-akait on 18 Jul 2018

Thank you for a quick fix!

hudochenkov picture hudochenkov on 18 Jul 2018
Was this page helpful?
0 / 5 - 0 ratings

Related issues

lukasoppermann picture lukasoppermann  路  7Comments
hacknug picture hacknug  路  5Comments
djmtype picture djmtype  路  6Comments
bernhardreiter picture bernhardreiter  路  7Comments
GrimLink picture GrimLink  路  7Comments