"use strict" followed by a comma could cause syntax error in Firefox 16.0.1 about uglifier HOT 4 CLOSED

Thanks for the bug report, although I believe this is something which needs to be addressed in UglifyJS. You can configure the comma separation using the seqs option, which is by default true.

In your Rails config:

config.assets.js_compressor = Sprockets::LazyCompressor.new {
  require 'uglifier'
  Uglifier.new(seqs: false)
}

I'll keep an eye on this and might change the default in Uglifier if the issue isn't fixed in either UglifyJS or Firefox.

from uglifier.

I'm having the same problem.

@symve - How did it turn out? Have you tried to set seqs false?

from uglifier.

Sorry, I didn't have time to try it out yet. Our JS code changed since, and as a result the problem in our minified code file went away.

from uglifier.

Ahah, I see, we got into the same situation but it came back yesterday... :)

I've just tried it and it seems to solve the problem with very, very little impact on the code size since at the end we're gzipping it anyway.

P.S.: and thanks for posting that issue by the way!

from uglifier.