patternfly / patternfly/react-data-view

Consistency: Add chip toolbar buttons to all DataView filters to match TreeViewFilter

オープン
#646 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

主要言語
TypeScript
スター
4
フォーク
21
PR マージ指標
30日以内にマージされた PR はありません

説明

Hello,

During a recent PR review, we noticed that the DataViewTreeViewFilter contains an extra button in the chip toolbar that isn't present in other data-view filters:

Image

This is, however, as per the examples I found in PatternFly documentation, the correct behaviour for filters:

Image

Could you please implement this behaviour to the rest of the data-view filters?

Many thanks for considering the change!


Jira Issue: PF-4106

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

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

はじめの一歩

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

調査の方向性

まず、DataViewTreeViewFilterを他のデータビューのフィルターと比較し、リンクされているPatternFlyのフィルター例を確認します。残りのフィルターを更新して、同じチップツールバーのボタン動作を提供するようにし、その後、すべてのデータビューフィルターのバリエーションで一貫性があることを確認します。

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

評価

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

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

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