Ehnanced wrapper to address AMD loader issues

millermedeiros / crossroads.js

JavaScript Routes

http://millermedeiros.github.com/crossroads.js/

1.44k stars 156 forks source link

Ehnanced wrapper to address AMD loader issues #40

Closed ghost closed 12 years ago

ghost commented 12 years ago

Unfortunately, when using the simplified CommonJS wrapping, it's not guaranteed the cross-browser support (i.e. RequireJS and Function.prototype.toString()). Specifying dependencies to define avoids that.

This commit only addresses wrapper.js, it doesn't include a complete rebuild.

millermedeiros commented 12 years ago

I ended up simplifying the wrapper a little bit and removing the name from the module as well (see 7101dd722ab8062088478f0484aab73a29a14103), thanks!