akiran / akiran/react-slick

bug: RTL show

オープン
#1,764 コメント 3 件 リアクション 3 件 担当者 0 名 GitHub で見る
bug
主要言語
JavaScript
スター
11.9k
フォーク
2.1k
PR マージ指標
30日以内にマージされた PR はありません

説明

### 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.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。