angular / angular/components

bug(material/chips): Chips should have larger touch targets

Đang mở
#29,511 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Accessibility area: material/chips P3
Ngôn ngữ chính
TypeScript
Star
25k
Fork
6.8k
Merge trung bình
1 ngày 8 giờ
Pull request đã merge (30 ngày)
91

Mô tả

### Is this a regression?

- [ ] Yes, this behavior used to work in the previous version

### The previous version in which this bug was not present was

_No response_

### Description

Even though icon button defaults to 40px, the touch target is still 48px from here: https://gith
ub.com/angular/components/blob/main/src/material/button/_button-base.scss#L126.

Screenshot 2024-07-29 at 2 34 16 PM
Screenshot 2024-07-29 at 2 35 10 PM

Chips has interactive icons buttons or the chip itself is interactive, it should also have a touch target of at least 48x48px to be more accessible

### Reproduction

https://material.angular.io/components/chips/examples

### Expected Behavior

Touch targets of at least 48x48px

### Actual Behavior

Smaller touch targets than 48x48px
Screenshot 2024-07-29 at 2 33 29 PM

### Environment

- Angular:
- CDK/Material:
- Browser(s):
- Operating System (e.g. Windows, macOS, Ubuntu):

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

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

Hướng nghiên cứu

Bắt đầu với các ví dụ về chip tại material.angular.io/components/chips/examples và so sánh kích thước của các chip tương tác và nút biểu tượng với mục tiêu 48px. Đọc các style được tham chiếu trong src/material/button/_button-base.scss, sau đó tìm phần triển khai chip tương ứng và các bài kiểm thử; công việc được hoàn tất khi các chip tương tác và biểu tượng của chúng cung cấp vùng mục tiêu cảm ứng ít nhất 48x48px.

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

Đánh giá

Công nghệ
angular, sass, typescript
Lĩnh vực
accessibility, frontend
Loại issue
Lỗi
Độ 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
45/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.