Restore Tab/Shift+Tab focus-exit test coverage
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 55/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 静か
- 技術スタック
- playwright, typescript
- 領域
- accessibility, testing
調査の方向性
Start with elements/pf-v5-accordion/test/pf-accordion.spec.ts and elements/pf-v5-search-input/test/pf-search-input.spec.ts, reviewing the existing keyboard-navigation tests and the four removed or weakened assertions. Run the relevant test files, then restore coverage for focus leaving each component through Tab or Shift+Tab using one of the listed approaches. Done means all four focus-exit cases are asserted again.
索引モデルが issue の本文から書いたものです。
説明
Context
PR #3121 migrated unit tests from Playwright to Puppeteer. Puppeteer cannot move focus outside the browser viewport via Tab/Shift+Tab, so four assertions that verified focus leaving a component were removed or weakened.
Removed/weakened assertions
- Accordion: Shift+Tab from first header exits accordion - removed entirely (no prior focusable element to land on)
- Accordion: Shift+Tab from last header exits accordion - removed entirely
- Accordion: Tab out then Shift+Tab back returns to panel link - removed the Shift+Tab return portion
- Search-input: Tab from expanded listbox defocuses combobox - changed from asserting nothing is focused to asserting listbox is closed
What to restore
All four cases test that focus leaves the component entirely via Tab/Shift+Tab. Options:
- Add Playwright e2e tests (
*.e2e.ts) for these keyboard navigation flows, since e2e tests still run on Playwright - Investigate whether Puppeteer's
page.bringToFront()or CDPInput.dispatchKeyEventwith proper target can simulate Tab-out behavior - Use a sentinel focusable element before/after the component in the test fixture to catch focus without needing to leave the page
Files affected
elements/pf-v5-accordion/test/pf-accordion.spec.tselements/pf-v5-search-input/test/pf-search-input.spec.ts
- 主要言語
- TypeScript
- スター
- 394
- フォーク
- 107
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
patternfly/patternfly-elements のほかの issue
-
bug
patternfly/patternfly-elements#3158 · 担当者 1 名 ·
-
patternfly/patternfly-elements#3136 · コメント 1 件 · 担当者 1 名 ·
-
docs
patternfly/patternfly-elements#3131 · 担当者 1 名 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 38/100
patternfly/patternfly-elements#3118 ·
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
patternfly/patternfly-elements#3110 ·
patternfly/patternfly-elements の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
ontola/atomic-server#1625 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
melgarafael/DeskcommCRM#1451 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
-
bug via-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
bot:ai-assisted component:compact-js status:untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
midnightntwrk/midnight-sdk#403 ·