Skip to content

Conversation

clamli
Copy link
Contributor

@clamli clamli commented Jan 30, 2024

This commit updates the single-select Mat toggle button group to use radio pattern. This way the exclusiveness of the selection can be conveyed to the screen reader users.

@clamli clamli requested a review from andrewseguin as a code owner January 30, 2024 22:05
@clamli clamli changed the title fix(MatButtonToggle): use radio pattern for single select Mat toggle button group fix(material/button-toggle): use radio pattern for single select Mat toggle button group Jan 30, 2024
@clamli clamli force-pushed the mat-button-toggle-group branch 2 times, most recently from fcd8a94 to 5c845a3 Compare February 1, 2024 00:44
@andrewseguin andrewseguin requested a review from zarend February 1, 2024 20:24
@clamli clamli force-pushed the mat-button-toggle-group branch from 5c845a3 to 98eb273 Compare February 1, 2024 22:28
@zarend zarend added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label Feb 2, 2024
Copy link

github-actions bot commented Feb 2, 2024

Deployed dev-app for 119797c to: https://ng-dev-previews-comp--pr-angular-components-28505-dev-r8qalkt6.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@clamli clamli force-pushed the mat-button-toggle-group branch 2 times, most recently from 644acdd to 3f7ccba Compare February 7, 2024 23:22
@clamli clamli force-pushed the mat-button-toggle-group branch from 3f7ccba to 119797c Compare February 7, 2024 23:42
@zarend
Copy link
Contributor

zarend commented Feb 9, 2024

We're considering using a different approach to make it easier to land, see #28548

@clamli clamli marked this pull request as draft February 9, 2024 21:19
@crisbeto
Copy link
Member

crisbeto commented Apr 18, 2024

Given that #28548 was merged, I'm assuming that we can close this PR.

@crisbeto crisbeto closed this Apr 18, 2024
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
dev-app preview When applied, previews of the dev-app are deployed to Firebase
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants