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

Duplicated Slides in Infinite Carousel v0.30.0+

Open rodrigo-arias opened this issue 1 year ago • 8 comments

Description

Encountering an unexpected behavior with react-slick versions 0.30.0 and above. When configuring a carousel with the same number of rows as items and enabling infinite scrolling, slides are duplicated, appearing in both the previous and next items.

Steps to reproduce

  1. Install react-slick v0.30.0 or a later version.
  2. Set up a carousel with an equal number of rows and items.
  3. Enable the infinite scrolling option.
  4. Observe the duplication of slides in both the previous and next items.

Feel free to see the difference by rolling back to v.0.29.0 with this CodeSandbox.

The change seems to have originated in #2329 in our case this is a visual breaking change.

rodrigo-arias avatar Jan 31 '24 20:01 rodrigo-arias

can you specify which file we have to work.

Lala-kumar avatar Feb 07 '24 05:02 Lala-kumar

Could you provide an example? The link isn't working?

rjsdnql123 avatar Apr 08 '24 17:04 rjsdnql123

@rodrigo-arias If the bug I identified is correct, it has been fixed in this pull request. Apologies for the delayed confirmation 😢

Since the maintainer has confirmed, we will soon deploy a new version!

https://github.com/akiran/react-slick/pull/2366

rjsdnql123 avatar Apr 11 '24 18:04 rjsdnql123

Using the latest version, this bug still exists

bfaulk96 avatar May 09 '24 15:05 bfaulk96

@akiran it looks like the fix was merged into master but never released, is there something stopping it from being released as a new version?

bfaulk96 avatar May 09 '24 15:05 bfaulk96

@rjsdnql123 Sorry for the late response. I couldn't find a beta, but I tried it by replicating #2366, and I don't see that it solves at least the bug I am reporting (rooted in #2329 and verified by setting the same number of rows and items).

rodrigo-arias avatar Jun 02 '24 23:06 rodrigo-arias

Any news about new release?

I've encountered the same issue and as a temporary solution, I've set adaptiveHeight: true. Somehow this setting hide cloned slides 😬

skrzepij avatar Jun 17 '24 12:06 skrzepij

@rodrigo-arias @skrzepij

This bug has been fixed. However, since the administrator has not deployed after merging, the bug fix has not been applied.

@akiran Could you please check on this?"

rjsdnql123 avatar Aug 10 '24 15:08 rjsdnql123