patternfly / patternfly/patternfly-react
Spike - Investigate options for updating a11y props being required or not
Chưa có ai nhận issue này.
- 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 enhancement or change
Spike for a discussion to have regarding https://github.com/patternfly/patternfly-react/issues/5397
Basically we need to decide whether:
-
we want to update certain a11y props to either be required (if currently optional) or make them optional (if currently required)
-
we want to add console warnings when an optional a11y prop is not passed in, but ideally should be (e.g. if an element doesn't have a visible text label, typically we should want an
aria-labeladded somewhere) -
leave things as-is - props will remain optional or required as they already are
-
some combination of all of the above
-
some props should have an (updated?) default value
Is this request originating from a Red Hat product team? If so, which ones and is there any sort of deadline for this enhancement?
Any other information?
Jira Issue: PF-108
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.
Hướng nghiên cứu
Bắt đầu bằng cách đọc issue #5397 của PatternFly được liên kết và xem xét các accessibility props bắt buộc và tùy chọn hiện tại trên các component bị ảnh hưởng. So sánh các tùy chọn được mô tả ở đây, bao gồm cảnh báo và giá trị mặc định. Được coi là hoàn tất khi các phương án thay thế, các props bị ảnh hưởng và một hướng được đề xuất đã được ghi lại để thảo luận.
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
- Loại issue
- Tính năng
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Ít trao đổi
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 35/100