nir9 / itemslide

JavaScript Touch Carousel Library with Zero Dependencies
https://itemslide.org
MIT License
647 stars 85 forks source link

Problem when using more than one itemslide per page #78

Closed eldegwy closed 3 years ago

eldegwy commented 6 years ago

i'm not able to set different options for different itemslides per the same page

i found the reason in dist/itemslide.js line 367 Was carousel.create($.extend(defaults, options), this); updated to carousel.create($.extend({},defaults, options), this);

Please make this fix :D Thank for your great itemslide

bjornbak commented 6 years ago

I stumbled upon the same problem and @eldegwy 's fix does solve it! Please add to master.

pamplo commented 6 years ago

The line in dist/itemslide.js moved to 364.

The solution worked for me and there is already a pull request with this change in place: #79

nir9 commented 3 years ago

Related to #86 , fixed therefore closing this one as well.