visionmedia / move.js

CSS3 backed JavaScript animation framework
http://visionmedia.github.com/move.js/
4.72k stars 685 forks source link

Make move.js YUI-compressor compatible. #11

Closed rstacruz closed 11 years ago

rstacruz commented 13 years ago

Right now, it's structured this way:

exports.Move = function Move() { ... }
Move.prototype.x = ...

This presents a problem as YUI compressor may mangle function Move() into a non-named function. We've encountered this problem using move.js with Jammit.

tj commented 13 years ago

damn, uglifyjs does the same, that's really annoying. IMO it's really wrong to alter code just for a compressor, YUI must have a setting or something. I'd rather just do:

function Move(){}

exports.Move = Move;

assuming it does not mess with function statements haha

rstacruz commented 13 years ago

YUI only has the following options:

yields commented 11 years ago

closing, if this is still an issue feel free to open :)