search

daxingplay / css-combo

combo css which import other css
MIT License
67 stars 30 forks source link

多个子元素选择器在同一组规则中出现时，若子元素选择器前无限定选择器，压缩后从第二个开始，其前面的空格会被去掉 #22

Closed librajt closed 10 years ago

librajt commented 10 years ago

如以下css

a :nth-child(2), a :nth-child(5), a :nth-child(9) {display: none;}

压缩生成的代码为

a :nth-child(2),a:nth-child(5),a:nth-child(9){display:none}

空格被压缩，导致元素选择错误。版本为0.3.2 对比jar的压缩工具生成的代码，是没有问题的

daxingplay commented 10 years ago

压缩是用的自带的压缩选项么？

librajt commented 10 years ago

是的，像这样的

combo.build(
    [
        path.resolve(__dirname, './a.css')
    ],
    path.resolve(__dirname, './a.min.css'),
    {
        compress: true
    },
    function(err, result){
        console.log(result);
    }
);

a.css内容就是上面写的那一段

daxingplay commented 10 years ago

谢谢您的反馈，在0.3.4版本中修复。

我个人的小建议：如果用grunt的话，建议用grunt-contrib-cssmin来压缩css，毕竟css-combo更专注打包:) 如果图省事，用css-combo直接压缩也没问题，有问题可以反馈~