Restore Tab/Shift+Tab focus-exit test coverage
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Ít trao đổi
- Công nghệ
- playwright, typescript
- Lĩnh vực
- accessibility, testing
Hướng nghiên cứu
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.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- TypeScript
- Star
- 394
- Fork
- 107
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của patternfly/patternfly-elements
-
bug
patternfly/patternfly-elements#3158 · 1 người được giao ·
-
patternfly/patternfly-elements#3136 · 1 bình luận · 1 người được giao ·
-
docs
patternfly/patternfly-elements#3131 · 1 người được giao ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 38/100
patternfly/patternfly-elements#3118 ·
-
[epic] Port v5 elements to v6 Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
patternfly/patternfly-elements#3110 ·
Tất cả issue của patternfly/patternfly-elements
Issue tương tự
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
-
Crush Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
ElementsProject/cln-application#167 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Quantco/pnpm-licenses#17 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100