search

yogthos / lein-asset-minifier

Leiningen plugin for CSS/Js asset minifcation
Eclipse Public License 1.0
41 stars 7 forks source link

Minifier is breaking some CSS, specifically calc(100% + 5px) #16

Open khmelevskii opened 6 years ago

khmelevskii commented 6 years ago 
.issue {
    width: calc(100% + 30px);
}

turns into .issue{width:calc(100%+30px)} instead of .issue{width:calc(100% + 30px)}

Notice the missing spaces

yogthos commented 6 years ago

CSS minification is handled by the YUI Compressor library, so it looks like it might be a bug there. Asset minifier is on the latest version, so one approach might be to look for an alternative library.

kengruven commented 5 years ago

Looks like yui/yuicompressor#268 to me. Strangely, that issue was recently closed, without comment, and with any recent changes.

yogthos commented 5 years ago

It looks like it might be worth looking at other options if the yuicompressor isn't actively maintained.