angular / angular/components

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

オープン
#29,511 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
Accessibility area: material/chips P3
主要言語
TypeScript
スター
25k
フォーク
6.8k
平均マージ
1日 8時間
マージ済み PR(30日)
91

説明

### 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):

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

material.angular.io/components/chips/examples のチップの例から始め、インタラクティブなチップとアイコンボタンの寸法を 48px の目標値と比較します。次に、参照されている src/material/button/_button-base.scss のスタイルを読み、その後、対応するチップの実装とテストを見つけます。インタラクティブなチップとそのアイコンが少なくとも 48x48px のタッチターゲットを提供すれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
angular, sass, typescript
領域
accessibility, frontend
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。