Skip to content

Gestures#25

Open
nono wants to merge 3 commits into
inspire-js:gh-pagesfrom
nono:gestures
Open

Gestures#25
nono wants to merge 3 commits into
inspire-js:gh-pagesfrom
nono:gestures

Conversation

@nono

@nono nono commented Oct 4, 2012

Copy link
Copy Markdown

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
Copy Markdown
Collaborator

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

nono commented Oct 30, 2012

Copy link
Copy Markdown
Author

Hi,

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

@LeaVerou

Copy link
Copy Markdown
Collaborator

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

nono commented Oct 30, 2012

Copy link
Copy Markdown
Author

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

@LeaVerou

Copy link
Copy Markdown
Collaborator

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

nono commented Oct 31, 2012

Copy link
Copy Markdown
Author

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

@nono

nono commented Oct 31, 2012

Copy link
Copy Markdown
Author

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

nono commented Nov 18, 2012

Copy link
Copy Markdown
Author

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

@nono

nono commented Jan 24, 2013

Copy link
Copy Markdown
Author

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

@LeaVerou

Copy link
Copy Markdown
Collaborator

Sorry not yet :( But thanks for reminding me.

@nono

nono commented Jun 30, 2013

Copy link
Copy Markdown
Author

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

@nono

nono commented Oct 11, 2013

Copy link
Copy Markdown
Author

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

@LeaVerou

Copy link
Copy Markdown
Collaborator

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

nono commented Oct 12, 2013

Copy link
Copy Markdown
Author

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

nono commented Dec 14, 2013

Copy link
Copy Markdown
Author

@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