easyrider7522 / yii2-bootstrap-touch-slider

Implementation of bootstrap-touch-slider from bootstrapthemes.co as Yii2 widget
MIT License
1 stars 0 forks source link

On Android (chrome + Firefox) buttons don't work. #2

Open lioPixiumDigital opened 5 years ago

lioPixiumDigital commented 5 years ago

Same thing when using chrome on MacOS in Developper / flat device toggled.

Are touch events handled?

easyrider7522 commented 5 years ago

Are they working on the original page: bootstrapthemes.co?

lioPixiumDigital commented 5 years ago

Yes, just tried, they are.

On 11 Dec 2018, at 15:03, easyrider7522 notifications@github.com wrote:

bootstrapthemes.co https://bootstrapthemes.co/demo/resource/BootstrapCarouselTouchSlider/

lioPixiumDigital commented 5 years ago

Do you have any issue when trying with Chrome in Dev mode, touch device simulation (as below), Your plugin seems to work on long touch (offers to open in new tab, see below), but no click...

bootstrap on the original page: bootstrapthemes.co https://bootstrapthemes.co/demo/resource/BootstrapCarouselTouchSlider/?. works.

On 11 Dec 2018, at 15:03, easyrider7522 notifications@github.com wrote:

Are they working on the original page: bootstrapthemes.co https://bootstrapthemes.co/demo/resource/BootstrapCarouselTouchSlider/?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/easyrider7522/yii2-bootstrap-touch-slider/issues/2#issuecomment-446094509, or mute the thread https://github.com/notifications/unsubscribe-auth/AffSxjpTf2lgjGyGKHTmHtj_lSmEIIMiks5u31ivgaJpZM4ZMxEn.

easyrider7522 commented 5 years ago

The slider uses the same JS as the original so the problem might be in the PHP... I already have Beta release of this port, it just needs to be prepared for GitHub and uploaded. In Beta the PHP is overworked a lot, so your problem may be gone as well, but I will try to check on it before the release. Will upload these days!

lioPixiumDigital commented 5 years ago

I used this as a quick fix image

easyrider7522 commented 5 years ago

You could also try to use TouchSwipe directly: in TouchSwipeAsset.php uncomment the direct link https://cdnjs.cloudflare.com/ajax/libs/jquery.touchswipe/1.6.18/jquery.touchSwipe.min.js and comment out the original jquery.touchSwipe.js (the $sourcePath and $publishOptions maybe are to be commented out also).

btw., smart solution of yours!

lioPixiumDigital commented 5 years ago

Thanks, btw, nice work with the Yii2 plug. An't wait for Alpha to move to Beta. 😉