knub / onehundredandtwenty

Course Planning System for HPI masters
15 stars 19 forks source link

Touch support #14

Closed philippotto closed 8 years ago

philippotto commented 10 years ago

There is currently no support for touch. I would suggest integrating hammer.js. Do you use a package manager like bower? Or do you commit the libs directly to the repo.

If you want, I can submit a PR.

knub commented 10 years ago

Unfortunately, there is no package manager (actually, bower did not exist back then). So, yes, the libs are committed directly to the repo.

Feel free to submit a PR. I have not used hammer.js, you think it integrates with the existing jQuery UI code?

philippotto commented 10 years ago

I'm not sure how well hammer.js harmonizes with jQuery UI. I will probably take a look at it at the weekend.

knub commented 8 years ago

@philippotto: Closing this as Won't fix, ok?

philippotto commented 8 years ago

Yes, feel free to do so. Sorry, that I didn't work on that. On Oct 22, 2015 1:51 AM, "Stefan Bunk" notifications@github.com wrote:

@philippotto https://github.com/philippotto: Closing this as Won't fix, ok?

— Reply to this email directly or view it on GitHub https://github.com/knub/onehundredandtwenty/issues/14#issuecomment-150153426 .