patternfly / patternfly/patternfly-react

Page: tabIndex default value (mainTabIndex=-1) which prevents keyboard scroll

Đang mở
#5,920 4 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

A11y Breaking change :boom:
Ngôn ngữ chính
TypeScript
Star
862
Fork
392
Merge trung bình
4 ngày 8 giờ
Pull request đã merge (30 ngày)
9

Mô tả

Describe the issue. What is the expected and unexpected behavior?
mainTabIndex=-1 as a default value prevents keyboard scroll. tlabindex should be removed. Having a tabindex on a container essentially breaks focus and keyboard scroll for all non-focusable containers below it. eg topology details side panel.

Any contributions to the main content should not have to deal with adding their own tabindex attribute to their scroll containers.

Above notes from the pr: https://github.com/openshift/console/pull/9140#issuecomment-854134911

workaround is to set the mainTabIndex to null.

Please provide the steps to reproduce. Feel free to link CodeSandbox or another tool.

https://github.com/openshift/console/pull/9140#issuecomment-8541349113

Is this a bug or enhancement? If this issue is a bug, is this issue blocking you or is there a work-around?
Bug
What is your product and what release version are you targeting?
OpenShift 4.9

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. 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.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu với pull request #9140 của PatternFly React được tham chiếu và issue comment được liên kết để xác định giá trị mặc định của mainTabIndex. Xóa hành vi tabindex mặc định, sau đó xác minh rằng việc cuộn bằng bàn phím hoạt động với các container không thể focus mà không cần workaround bằng mainTabIndex.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
react, typescript
Lĩnh vực
accessibility, frontend
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.