akiran/react-slick

slick-cloned slides visible if slidesToDisplay > slide count

Open

#1 569 ouverte le 11 juin 2019

Voir sur GitHub
 (4 commentaires) (2 réactions) (0 assignés)JavaScript (11 386 stars) (2 109 forks)batch import
good first issue

Description

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.

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic ReactUnderstanding of react slick
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
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.