akiran / akiran/react-slick

bug: RTL show

Offen
#1,764 3 Kommentare 3 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
bug
Vorherrschende Sprache
JavaScript
Sterne
11.9k
Forks
2.1k
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

### Describe
I think in RTL, from right to left, it's 123, not 312

### CodeSandbox
https://codesandbox.io/s/react-slick-playground-lk4i7?file=/index.js

https://github.com/akiran/react-slick/blob/80eb208cb7454cc63621f3bcf07a2b8c32999afa/src/track.js#L183
It may be fix in this place
```diff
--preCloneSlides.concat(slides, postCloneSlides).reverse();
++preCloneSlides.concat(slides.reverse(), postCloneSlides.reverse());
```

And `preCloneSlides` is not the last node, but the first node.

I hope this will be fix.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.