Skip to content

MooTools compatibility - #3

Closed
ryanmitchell wants to merge 1 commit into
hammerjs:masterfrom
thoughtco:master
Closed

ryanmitchell wants to merge 1 commit into
hammerjs:masterfrom
thoughtco:master

Conversation

@ryanmitchell

Copy link
Copy Markdown

Hello

Great library guys - really well written and structured.

We use MooTools here, and I needed to get the library working in it so we weren't running jQuery and MooTools side by side. Turned out to be just an hours work - which is amazing.

I've forked and amended the code so you can include it if you want - didn't change any of the original files. I haven't updated all the demos, just the ones I wanted, and theres lots I could do to make the hammer.js file more MooTools-ish (more class orientated), but hey it works and its great.

Thanks,
Ryan

@Aratramba

Copy link
Copy Markdown
Contributor

Hi Ryan,

Thanks for your comment. I guess we'd rather have a clean 'vanilla' hammer.js, with mootools, jquery, etc extensions alongside. We're working on removing jquery from the core and will probably later add an extension which handles the extra event triggers. Thanks for your work though, we'll use it as a reference.

Cheers,

Arjen

@ryanmitchell

Copy link
Copy Markdown
Author

No worries at all - thats absolutely the right way to go.
I look forward to seeing it - give me a shout if you need any help at all.

@jaap3 jaap3 closed this Mar 29, 2012
runspired added a commit that referenced this pull request Jul 12, 2016
feat(ES6 classes): adds the ES6 class syntax and functionality.
pzuraq pushed a commit to pzuraq/hammer.js that referenced this pull request Jul 12, 2016
chore(dependencies): pulls perf data structures out into the perf-pri…
daybrush added a commit to daybrush/hammer.js that referenced this pull request Oct 29, 2018
fix(hammerjs): fix window for nodejs
Close hammerjs#3
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.

3 participants