Disable warning for TabPanel count of 0
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 38/100
- issue の種類
- 機能追加
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript, react
- 領域
- frontend
調査の方向性
まず、issue で説明されている同数警告を出力する Tabs のバリデーションを見つけます。TabPanel の数が 0 の場合にどのように処理しているかを確認し、その場合に限って警告を抑制するよう動作を更新し、他の数の不一致では引き続き警告が出ることを検証します。
索引モデルが issue の本文から書いたものです。
説明
Hello,
I am using react-tabs with with success in my app.
In some places though, I use it with no panel at all. In that case, it is used to change a property on another component.
Stripped down example to illustrate:
const [page, setPage] = useState(0);
<Tabs selectedIndex={page} onSelect={setPage}>
<TabList>{ /* one Tab per page here */ }</TabList>
</Tab>
<DocumentView url={url} page={page} />
This works perfectly.
However, react-tabs complains that
There should be an equal number of 'Tab' and 'TabPanel' in
Tabs
It then continues happily and everything works fine.
- Do you see any issue with this use case?
- If not, would it be possible to silence that warning when the number of
TabPanels is exactly 0?
- 主要言語
- JavaScript
- スター
- 3.1k
- フォーク
- 455
- 平均マージ
- 5時間 9分
- マージ済み PR(30日)
- 2
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
reactjs/react-tabs のほかの issue
-
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
reactjs/react-tabs#584 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
reactjs/react-tabs#581 · コメント 3 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
reactjs/react-tabs#571 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
reactjs/react-tabs#525 · コメント 1 件 · リアクション 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
reactjs/react-tabs#521 · コメント 3 件 ·
reactjs/react-tabs の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
avniproject/avni-client#2135 ·
-
automated broken-link
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
agent/security hive/hosted-available-lke648397-260827-5n31 security
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
babalae/bettergi-scripts-list#3674 ·
-
A-Release-Notes C-Editing D-Modest S-Ready-For-Implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
bevyengine/bevy-website#2595 ·