search

webber12 / eFilter

Фильтрация ресурсов по TV. MODxEvo
16 stars 19 forks source link

Touch support for range slider #5

Closed Nicola1971 closed 8 years ago

Nicola1971 commented 9 years ago

jQuery UI range slider does not work on mobile devices. Can you add jquery-ui-touch-punch? https://github.com/furf/jquery-ui-touch-punch

http://fellowtuts.com/jquery/jquery-ui-sliders-on-touch-devices/

webber12 commented 9 years ago

done https://github.com/webber12/eFilter/commit/4142d6f359b5bae4082c74f560618d3af8e52a4d & https://github.com/webber12/eFilter/commit/218073dfbabdc2766b15997bb607606e6aae5dd4

Nicola1971 commented 9 years ago

Does it works for you? I've tested on my ipad4 and my Android Phone, but does nothing :(

webber12 commented 9 years ago

Sorry, ones more small fix - https://github.com/webber12/eFilter/commit/e4273beb72ab4ef789dda225f124a07d397cb6f8 :) I`ve tested on my android phone - now it works correctly for me.

Nicola1971 commented 9 years ago

Ok, Now it's more touch friendly: I can "tap" to set the slider range but i can't "drag" like:

http://refreshless.com/nouislider/

http://ionden.com/a/plugins/ion.rangeSlider/en.html

Is jQuery UI used in other parts of the plugin as well the rangeslider?

webber12 commented 9 years ago

In this project jQuery UI is used only for slider. But in my tests on andro-phone both "tapping" and "dragging" work correctly. May be some js-cashing problem or ipad/ios-only problem in your case?