search

kenwheeler / slick

the last carousel you'll ever need
kenwheeler.github.io/slick
MIT License
28.54k stars 5.89k forks source link

Different .min.js and .js #3649

Open sanathkash opened 5 years ago

sanathkash commented 5 years ago

When I used slick in my most recent project and when my code was integrated with AEM we noticed that the slick.min.js and slick.js files had different code for the slick load functionality. The slick.min.js file was behaving as expected and the slick.js was not. So we just unminified the slick.min.js and used that instead of slick.js. Please look into this issue and resolve it at the earliest so that others will not face the same issue.

RaymondBakker commented 5 years ago

Same the minified version works, but the other does not.

KennethVerbeure commented 5 years ago

we see the same issue here... slick renders elements differently (adding extra div elements in slick.js but not in slick.min.js). this breaks our code because we expect the elements to be rendered in dev/production builds in the same way

AndreyYevtukhov commented 4 years ago

Same here.

slick.js adds additional divs around slides, slick.min.js - does not.

flack commented 4 years ago

it looks like the min.js is stuck at version 1.8.0, so it's missing a few commits. So as a workaround, you can probably just minify slick.js yourself

fgm commented 4 years ago

Reverse situation here: slick.min.js breaks centerMode, while slick.js centerMode works: the minified version add a width: NNNpx;transform: translate3d(-NNNpx, 0px, 0px). The value of NNN is width - something_not_identified.

Codename-Duchess commented 3 years ago

Similar to fgm, slick.min.js isn't working correctly for me, but slick.js is.

For info, my issue was with asNavFor not syncing when scrolling. It was scrolling about 20% further than expected, so by the time image 5 was displayed in the top slider, the thumbnail had already scrolled past.

darrencoutts118 commented 1 year ago

Suffering from the same issue with Slick. Is there any update on this as it is almost a 5-year-old issue.