akiran/react-slick

appendDots is overriding className declarations

Open

#1 533 ouverte le 11 avr. 2019

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

Description

with this configuration appendDots: dots => <div className="myClass"><ul>{dots}</ul></div>,

div.myClass is reasonable to expect for, div.slick-dots is resulting tough.

Guide contributeur

Stack technique
react
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.
mostly clear
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Investigate the source code of react slick, likely in the Dots component (e.g., src/dots.js or similar) to understand how `appendDots` merges classes. The issue suggests that the component always adds 'slick dots' class, but the user's custom className should be preserved. Look for where className is set and consider using a library like classnames to merge. Check comments in the issue for any proposed solutions or maintainer responses.