Allow ReactiveList to optionally render a sort label when using sortOptions
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
- issue の種類
- 機能追加
- 明瞭さ
- 説明が足りない
- 活発さ
- 停滞
- 技術スタック
- javascript, react
- 領域
- frontend
調査の方向性
まず ReactiveList の sortOptions と renderSortOptions の実装を見つけ、次にその renderResultStats prop と innerClass の動作を比較します。label がカスタムの render prop とデフォルトの showSortLabel オプションのどちらを使うべきかを判断し、関連する React テストを確認する前に期待されるレンダリング動作を定義します。
索引モデルが issue の本文から書いたものです。
説明
Affected Projects
React
Is your feature request related to a problem? Please describe.
In our current application we make use of the ReactiveList sorting however this only produces the select dropdown and has no way to tap into the render process to add a custom label (i.e. Material UI Typography) named "Sort By".
Describe the solution you'd like
A few options would work. Simple option is to add a renderSortLabel prop which render the custom label and this is then referenced in the renderSortOptions function and if set renders the label before the select.
Or have a prop named showSortLabel that renders a default "Sort By" label.
Alternative option could be to replicate that of the renderResultStats function where a custom render function is passed in as a prop and would handles the complete generation of the sort although this does is make it a lot more complex due to the handleSortChange handler.
Describe alternatives you've considered
Will start looking the innerClass to see if it is possible to tap into the sortOptions this and use the ::before pseudo-element.
Additional context
- 主要言語
- JavaScript
- スター
- 4.9k
- フォーク
- 479
- 平均マージ
- 4日 10時間
- マージ済み PR（30日）
- 4
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
appbaseio/reactivesearch のほかの issue
-
ci: add Node.js version matrix, caching, and lint step to test workflow対応中かも @Talha12Shiekh が 71 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
appbaseio/reactivesearch#2322 ·
メンテナーはふだん 1 日以内に返信
-
chore: add Dependabot config for automated dependency updates対応中かも @Talha12Shiekh が 71 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
appbaseio/reactivesearch#2321 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
security: user-controlled URL params injected directly into Redux store via JSON.parse対応中かも @Talha12Shiekh が 71 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
appbaseio/reactivesearch#2319 ·
メンテナーはふだん 1 日以内に返信
-
refactor: consolidate 12 near-identical SVG icon components into one pattern対応中かも @Talha12Shiekh が 73 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
appbaseio/reactivesearch#2324 ·
メンテナーはふだん 1 日以内に返信
-
refactor: SearchBox.js is 2692 lines — extract subcomponents for maintainability対応中かも @Talha12Shiekh が 71 日前に担当しました。 オープン
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
appbaseio/reactivesearch#2323 ·
メンテナーはふだん 1 日以内に返信
appbaseio/reactivesearch の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
NaturalIntelligence/fast-xml-parser#888 · コメント 1 件 ·
メンテナーはふだん 2 日以内に返信
-
bug callouts regression revealjs
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
quarto-dev/quarto-cli#15014 ·
メンテナーはふだん 1 日以内に返信
-
check:passed feeds:remove
難易度 1/5 1時間未満 初心者へのやさしさ 65/100
iptv-org/database#37176 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 9 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
hawk-digital-environments/HAWKI#443 ·
メンテナーはふだん 1 日以内に返信
-
documentation v2
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
modelcontextprotocol/python-sdk#3662 ·
メンテナーはふだん 1 日以内に返信