patternfly / patternfly/patternfly-react

Checkbox/Radio - allow flow content as body

オープン
#9,111 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Pinned
主要言語
TypeScript
スター
862
フォーク
392
平均マージ
4日 8時間
マージ済み PR(30日)
9

説明

Is this a new component or an extension of an existing one?

Existing components, Checkbox and Radio.

Describe the feature

To allow flow content for their body prop. Currently it's wrapped in a span element, which only admits phrasing content.

Are there visuals for this feature? If applicable, please include examples for each state and for varying widths

I don't think so.

Any other information?

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

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

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

Checkbox と Radio コンポーネントから始め、issue で参照されている isLabelWrapped の動作を含め、それらの body prop がどのようにラップされているかを確認します。既存の label の動作を壊さずに flow content をサポートできるか判断します。完了条件は、両方のコンポーネントが意図したコンテンツを受け入れること、または Checkbox のドキュメントで body のコンテンツは phrasing content でなければならないと明確に記載されていることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
react, typescript
領域
frontend
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

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

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