akiran/react-slick

slick-cloned slides visible if slidesToDisplay > slide count

Open

#1.569 geöffnet am 11. Juni 2019

Auf GitHub ansehen
 (4 Kommentare) (2 Reaktionen) (0 zugewiesene Personen)JavaScript (11.386 Stars) (2.109 Forks)batch import
good first issue

Beschreibung

In recent versions, if slidesToShow is set to a value greater than the number of slides, all of the slick-cloned slides will be visible.

To replicate: in the PlayGround (https://codesandbox.io/s/ppwkk5l6xx) update the settings:

var settings = {
      dots: true,
      slidesToShow: 5
    };

Version 0.21.0 does not behave this way. With version 0.21.0 if the slidesToShow is greater than the number of slides, it does not create the slick-cloned slides.

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic ReactUnderstanding of react slick
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40
Research-Richtung
Investigate the logic that generates slick cloned slides in react slick, specifically in the component that handles slidesToShow. Compare current behavior with version 0.21.0 where the clones were not created when slidesToShow exceeds slide count. The relevant code is likely in the initialization or rendering of slides. Check for a condition that should prevent cloning when slide count is less than slidesToShow.