react-slick icon indicating copy to clipboard operation
react-slick copied to clipboard

Removing unnecessary div wrapper on slides

Open akiran opened this issue 4 years ago • 5 comments

Currently, react-slick wraps each slide with div, which deviates from the slick-carousel implementation. So we will remove unnecessary div after fixing other pending issues in this library. Hopefully, we will make a release with this fix in a month or two.

This would be breaking change. When writing CSS be aware that div wrapper on the slide will be removed in future.

Using this issue for tracking progress and discussions related to this change.

akiran avatar Aug 20 '20 19:08 akiran

Would love to see this completed!

AlexMorrisonDev716 avatar Apr 01 '21 00:04 AlexMorrisonDev716

Is this going to be resolved? Just been around in circles on all the issues referring to this and can't find any mentions of a fix, nor any workarounds that solve the problem for us.

jakepm avatar Sep 29 '21 14:09 jakepm

I currently face the same problem. Has the react-slick team included the fix into any releases? With the presence of the unnecessary div, the horizontal margin between slides gets removed unexpectedly.

honghainguyen777 avatar Jan 24 '22 13:01 honghainguyen777

Any update? I'm facing this issue on v0.27.12

meezin avatar Feb 09 '22 19:02 meezin

@akiran Should I work on this?

ashutosh887 avatar Jun 15 '23 09:06 ashutosh887