patternfly / patternfly/patternfly-react

Button - update console error for accessible name and move aria-expanded location

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

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

Breaking change :boom: PF Team
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
Followup to some comments left in https://github.com/patternfly/patternfly-react/pull/11861/

  • The console error with the isSettings || isHamburger || isFavorite conditional for the accessible name (line 164 in that PR) should be updated to remove that conditional. We should be throwing an error for any button that doesn't have children, aria-label, or aria-labelledby
  • The aria-expanded attribute (line 239) should be moved below the {...props} so that it won't be overridden by a manual aria-expanded being passed in
  • Several tests related to the above points will need to be updated in some fashion as well

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?

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 bằng cách xem xét các thay đổi được tham chiếu trong pull request #11861, đặc biệt là lỗi console accessible-name quanh dòng 164 và cách xử lý aria-expanded quanh dòng 239. Xác định các bài kiểm thử Button liên quan được đề cập trong issue, cập nhật chúng theo hành vi được yêu cầu và xác nhận rằng cách xử lý lỗi và thuộc tính phù hợp với các yêu cầu đã nêu.

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
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
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
55/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.