Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Make the plugin Common JS and Require JS Compatible (AMD) #55

Closed
tabouassaleh opened this Issue May 3, 2012 · 3 comments

Comments

Projects
None yet
2 participants

For some boiler plate to add support for AMD, see https://github.com/umdjs/umd.

Owner

azoff commented May 3, 2012

I think you're under the impression I care about supporting AMD. I Intend no disrespect - but if you want AMD support, feel free to start your own fork of overscroll.

The best justification I can give you is that AMD support is tangential to making overscroll great. I look at AMD as a goal that can live along side Overscroll, without their fates being intertwined.

  • Jon

On May 2, 2012, at 6:34 PM, Tony Abou-Assalehreply@reply.github.com wrote:

For some boiler plate to add support for AMD, see https://github.com/umdjs/umd.


Reply to this email directly or view it on GitHub:
azoff#55

@azoff azoff closed this May 3, 2012

@azoff:

It was just a friendly feature request since I've seen several projects providing both versions. Thanks for clarifying your position on this topic.

Could you add a comment to this effect to the readme or the wiki so other people who might be looking for an AMD version would know what (not) to expect?

Cheers,
TAA

Owner

azoff commented May 3, 2012

No problem, consider it on my todo list.

On May 2, 2012, at 9:26 PM, Tony Abou-Assalehreply@reply.github.com wrote:

@azoff:

It was just a friendly feature request since I've seen several projects providing both versions. Thanks for clarifying your position on this topic.

Could you add a comment to this effect to the readme or the wiki so other people who might be looking for an AMD version would know what (not) to expect?

Cheers,
TAA


Reply to this email directly or view it on GitHub:
azoff#55 (comment)

@azoff azoff reopened this May 3, 2012

@azoff azoff closed this May 17, 2012

@azoff azoff referenced this issue May 17, 2012

Merged

1.6.3 Changes #61

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