akiran/react-slick

appendDots is overriding className declarations

Open

#1,533 opened on 2019年4月11日

GitHub で見る
 (9 comments) (0 reactions) (0 assignees)JavaScript (2,109 forks)batch import
good first issue

Repository metrics

Stars
 (11,386 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

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.

コントリビューターガイド

調査方針
react slick のソースコードを調査し、おそらく Dots コンポーネント内（例：src/dots.js など）で `appendDots` がクラスをどのようにマージするかを理解します。この問題は、コンポーネントが常に 'slick dots' クラスを追加するが、ユーザーのカスタム className は保持されるべきであることを示唆しています。className が設定されている場所を探し、classnames のようなライブラリを使用してマージすることを検討してください。issue のコメントを確認し、提案された解決策やメンテナの返信がないか調べてください。
技術スタック
reactjavascript
領域
frontend
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
React
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70