Breaks Irregular CSS content

[paynecodes]

I have documented the issue here.

• Breaks irregular css content gruntjs/grunt-contrib-cssmin#44

This seems to be the problem area, but my regex chops are no where near capable of fixing it, unfortunately.

• https://github.com/GoalSmashers/clean-css/blob/master/lib/clean.js#L122

[GoalSmashers]

It actually should not even get to this point since we escape all strings to prevent such situations. Here it doesn't work so consider your challenge accepted!

[paynecodes]

@GoalSmashers Thanks!

Here is a correction:

Before clean-css

  body::before {
    content: '{ "current" : "small", "all" : ["small"], "position" : 0 }';
    display: none;
  }

Notice, the quotes (") in ["small"] should be safe, but don't seem to be.

[GoalSmashers]

@jpdesigndev - you identified the place correctly as we actually strip quotation from CSS attributes BEFORE escaping all text. And ["small"] was matched as an attribute but it shouldn't be (cause it's an invalid CSS).

It is fixed now and 1.1.6 is out. Thanks for bringing it into our attention! 

[paynecodes]

@GoalSmashers Thank you! Your awesome. I haven't been able to test on my end yet, because I can't figure out how to update a dependencie's dependency as documented here.

[paynecodes]

@GoalSmashers Got it. I had to remove the modules in my node_modules folder, and reinstall them with npm install. I'm thinking it's because of this https://github.com/isaacs/npm/issues/1341.

[GoalSmashers]

Glad you got it. 👍 Seems like that's the problem with npm install.