nolimits4web / swiper

Most modern mobile touch slider with hardware accelerated transitions
https://swiperjs.com
MIT License
39.88k stars 9.74k forks source link

Memory leak when changing browser tabs #7246

Closed cwe-spb closed 11 months ago

cwe-spb commented 11 months ago

Check that this is really a bug

Reproduction link

https://swiperjs.com/demos/110-slides-per-view/core

Bug description

On any version of the package, if you open it in any browser (checked on ios + safari \ chrome), let's say https://swiperjs.com/demos/110-slides-per-view/core it will work perfectly. The swipes will be smooth. If you try to switch to another tab and go back to the swiper demo, you will see that there is a noticeable degradation of the scroll animation. There is a direct dependence on the complexity of the site where you switch to and the strength of the scroll swipe degradation

Expected Behavior

I have seen that tickets are being closed with information that there is a leak inside swiper, but the expected behavior is the absence of degradation

Actual Behavior

obvious, visible problems with the smoothness of animation

Swiper version

8-9-10-11

Platform/Target and Browser Versions

any

Validations

Would you like to open a PR for this bug?

cwe-spb commented 11 months ago

i founded just 1 slider without this bug. https://naver.github.io/egjs-flicking

i think trouble into swiper core

cwe-spb commented 11 months ago

I found the differences between the sliders. flicking followed the path of smoothly changing the transform parameter to js. Sniper's animation is implemented via translate3D with instant value change and translate-duration

for ex: https://ssense.github.io/vue-carousel/examples/ have same css work. and have some bug. translate3D work bad.

How i can change it at transition: translate()?

nolimits4web commented 11 months ago

Can not replicate it whatever i try