Skip to content

Conversation

JanPetr
Copy link
Contributor

@JanPetr JanPetr commented Nov 7, 2017

Fixes #902

@sam327, could you give it a try if it fixes your issue?

@sam327
Copy link

sam327 commented Nov 7, 2017

Thank you! Put it into a dev site, and my initial testing looks good. Works as I'd expect on my older ipad.

Throwing it at the rest of my office, as they've got a variety of apple devices, and we'll see how it goes.

@JanPetr
Copy link
Contributor Author

JanPetr commented Nov 7, 2017

Thanks for coming back to me @sam327 💯
Let me know if it's OK for your teammates as well and I'll merge :)

@sam327
Copy link

sam327 commented Nov 7, 2017

So, this looks good to us. Tested on a half-dozen apple devices, and a couple non-apple ones (for good measure).

@JanPetr JanPetr merged commit 7237ff2 into develop Nov 8, 2017
@JanPetr JanPetr deleted the fix/mobile_hover branch November 8, 2017 12:18
@JanPetr
Copy link
Contributor Author

JanPetr commented Nov 8, 2017

Thanks for your help @sam327! 💯 It's been merged and will be part of the next release.

@Haroenv
Copy link

Haroenv commented Nov 8, 2017

This is amazing! Let's see how to incorporate this in autocomplete.js itself

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