search

dynamicweb / swiffy-slider

Super fast carousel and slider with touch for optimized websites running in modern browsers.
MIT License
250 stars 29 forks source link

Looping end to start not working on Webkit Browsers #92

Closed kurtjmeyoung closed 4 months ago

kurtjmeyoung commented 7 months ago

Describe the bug Currently the infinite loop does not work to go back from last slide to the first slide. However it is working the other way around, from first slide to last slide. It seems this is only a problem on Webkit based browsers. It doesn't work on Chrome, Edge, Brave but does work on Firefox.

To Reproduce Steps to reproduce the behavior:

  1. You can simply test it on the configuration page: https://swiffyslider.com/configuration/

Expected behavior The infinite loop should also work on Webkit based browsers.

Desktop (please complete the following information):

Smartphone (please complete the following information):

GarrettL commented 5 months ago

Having the same issue as well

nicped commented 5 months ago

I cannot reproduce in any webkit browsers in any viewports...

https://github.com/dynamicweb/swiffy-slider/assets/3715307/3f067681-d301-4eba-a0fa-c568f60d5e7f

Can you show me an example?

kurtjmeyoung commented 5 months ago

I also can't reproduce it anymore now. It is now working for me both on https://swiffyslider.com/configuration/ as well as on my own page. Don't know what happened. Maybe something has changed on the chrome side of things?

nicped commented 5 months ago

Maybe... There have been some very very very specific issues with precise viewport widths - +/- 1px that could cause this, mostly seen in FF though. It could be a change related to that in recent updates.

I have also seen this when Swiffyslider is being inserted into an implementation where the containing dom element is not sizing 'correct' and confuses the scroll position.