Skip to content

Conversation

@Bantam
Copy link

@Bantam Bantam commented Nov 3, 2013

Improves requirejs support of tinycolor by supporting namespaces in the r.js optimizer as described here http://requirejs.org/docs/faq-optimization.html

Improves the requirejs support of tinycolor by supporting namespaces in the r.js optimizer as described here http://requirejs.org/docs/faq-optimization.html
@bgrins
Copy link
Owner

bgrins commented Nov 3, 2013

Interesting, I've had some weird stuff happen with all the different ways to import the script (like bgrins/spectrum#113 and bgrins/spectrum#41).

Since you seem to know something about this, maybe you would be a good person to ask - what is the current "best" way to expose the lib to all the different environments? Here is what we are doing right now - https://github.com/bgrins/TinyColor/blob/master/tinycolor.js#L950-L961. And will this change still work with older require/amd versions?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants