Button - update console error for accessible name and move aria-expanded location
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- react, typescript
- Lĩnh vực
- accessibility, frontend
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.
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 || isFavoriteconditional 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-expandedattribute (line 239) should be moved below the{...props}so that it won't be overridden by a manualaria-expandedbeing 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?
- 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
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-react
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12651 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
patternfly/patternfly-react#12637 ·
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
patternfly/patternfly-react#12617 · 1 bình luận ·
-
PF Team
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12610 · 1 bình luận · 1 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
patternfly/patternfly-react#12600 ·
Tất cả issue của patternfly/patternfly-react
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