Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gestures #25

Open
wants to merge 3 commits into
base: gh-pages
Choose a base branch
from
Open

Gestures #25

wants to merge 3 commits into from

Conversation

nono
Copy link

@nono nono commented Oct 4, 2012

Hi,

I've added support for gestures on smartphones (see #3). In therory, it should work on firefox, android, ios safari and blackberry if I believe http://caniuse.com/#feat=touch but I don't have the devices to test them all.

@LeaVerou
Copy link
Owner

Hi nono,

Sorry for the delay in getting back to you. This looks great and it's a long overdue feature, thank you!! Do you have any demo anywhere so that I could test it a bit before I merge?

Thanks!

@nono
Copy link
Author

nono commented Oct 30, 2012

Hi,

no problem, you can test it on http://blog.menfin.info/Presentations/20121018_Mobile_apps/ .

@LeaVerou
Copy link
Owner

Hey, thanks. Which devices have you tried this on? I just tried it on my iPhone and I can't seem to get it to work.

@nono
Copy link
Author

nono commented Oct 30, 2012

Firefox and chrome on an android device, and chromium on my desktop by emulating touch events.

@LeaVerou
Copy link
Owner

It doesn't seem to be working at all on iOS. If it will help, I can send you my browserstack info so you can test it for yourself. :)

@nono
Copy link
Author

nono commented Oct 31, 2012

Yup please, it's better than nothing ;-)

@nono
Copy link
Author

nono commented Oct 31, 2012

Well, it works but it was very difficult to trigger the gestures on iphone. I've changed some parameters and I hope it's better now. You can try the new version on http://www.menfin.info/csss/. Does it work on a real device?

By the way, the current geasture for moving to the next slide is a move from left to right. And right to left for moving to the previous slide. But maybe, the inverse is better. What do you think?

@nono
Copy link
Author

nono commented Nov 18, 2012

@LeaVerou: I'm sure you are busy, but if you find some time to test it, please let me know.

@nono
Copy link
Author

nono commented Jan 24, 2013

Hi Lea, did you find the time to test it?

@LeaVerou
Copy link
Owner

Sorry not yet :( But thanks for reminding me.

@nono
Copy link
Author

nono commented Jun 30, 2013

Hi @LeaVerou, this is a new reminder for this pull request. I hope you will have some time for it this time.

@nono
Copy link
Author

nono commented Oct 11, 2013

@LeaVerou this is yet another remainder. Maybe you have some time for it now.

@LeaVerou
Copy link
Owner

Shit, I’m so, so sorry I haven't reviewed this yet. :(

Btw there are merge conflicts now it seems. Could you please pull in any changes and resolve the merge conflicts? There shouldn't be many.

Again, thanks so much for this and I'm sorry I'm being so busy :(

@nono
Copy link
Author

nono commented Oct 12, 2013

Don't worry, I know what it is to have a lot of stuff ;)

I've rebased my commits and fix the conflicts (they were trivials). So, you should be able to merge them without pain.

@nono
Copy link
Author

nono commented Dec 14, 2013

@LeaVerou a quick remainder

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