nolimits4web / swiper

Most modern mobile touch slider with hardware accelerated transitions

Home Page:https://swiperjs.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

swiper.destroy() not working when createElements is enabled

samzabala opened this issue · comments

Check that this is really a bug

  • I confirm

Reproduction link

https://codesandbox.io/p/sandbox/swiper-stupid-38lq4g?file=%2Findex.html

Bug description

swiper.destroy() not working when createElements is enabled. wrapperEl is created dynamically when set to true, but destroy breaks scripts and returns fatal error when this is the setup

Error:
0:4290 Uncaught TypeError: Cannot read properties of undefined (reading 'removeAttribute')

Demo:
https://codesandbox.io/p/sandbox/swiper-stupid-38lq4g?file=%2Findex.html

Offending line was found ion the destroy method :

wrapperEl.removeAttribute('style');

could be fixed if there is a condition to check if it exists first

if(wrapperEl) wrapperEl.removeAttribute('style');

please help. thank

Expected Behavior

destroy the slider

Actual Behavior

destroys the site

Swiper version

~11.0.5 - 11.1.1 and 11.1.2 checked

Platform/Target and Browser Versions

Chrome/Edie/Webkit Mac OS

Validations

  • Follow our Code of Conduct
  • Read the docs.
  • Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
  • Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

  • I'm willing to open a PR

t0ggles-create swiper

Task nolimits4web/SWIPER-133 was created

t0ggles task SWIPER-133

Task nolimits4web/SWIPER-133 status changed to Done