Skip to content

Conversation

@adolgachev
Copy link
Contributor

@adolgachev adolgachev commented Dec 5, 2025

Fixes a11y issues when no label is used, but the label wrapper (with an onclick) is still present. This can lead to an extra interactive tab-stop depending on the screen reader used.

The label element is now set to display: none if it is empty.

@angular-robot angular-robot bot added the detected: feature PR contains a feature commit label Dec 5, 2025
@adolgachev adolgachev changed the title feat(material/slide-togge): Add option to hide label feat(material/slide-toggle): Add option to completely hide label Dec 5, 2025
@adolgachev adolgachev added Accessibility This issue is related to accessibility (a11y) action: review The PR is still awaiting reviews from at least one requested reviewer target: minor This PR is targeted for the next minor release labels Dec 5, 2025
@adolgachev adolgachev marked this pull request as ready for review December 5, 2025 19:34
@pullapprove pullapprove bot requested review from mmalerba and ok7sai December 5, 2025 19:34
@adolgachev adolgachev added the action: global presubmit The PR is in need of a google3 global presubmit label Dec 5, 2025
@mmalerba mmalerba removed their request for review December 5, 2025 19:40
@pullapprove pullapprove bot requested a review from tjshiu December 5, 2025 19:40
@adolgachev adolgachev requested a review from crisbeto December 8, 2025 17:18
@adolgachev adolgachev added dev-app preview When applied, previews of the dev-app are deployed to Firebase docs: preview When applied, a preview of the documentation site is deployed to Firebase labels Dec 8, 2025
@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Deployed dev-app for e93609f to: https://ng-dev-previews-comp--pr-angular-components-32480-dev-8wd8hfe4.web.app

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

@github-actions
Copy link

github-actions bot commented Dec 8, 2025

Deployed docs-preview for 97fc2d0 to: https://ng-dev-previews-comp--pr-angular-components-32480-docs-jpaocuag.web.app

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

@adolgachev adolgachev removed the request for review from ok7sai December 10, 2025 00:59
@adolgachev adolgachev changed the title feat(material/slide-toggle): Add option to completely hide label fix(material/slide-toggle): Fix a11y issues by hiding label when it has no content. Dec 10, 2025
@adolgachev adolgachev added target: patch This PR is targeted for the next patch release requires: TGP This PR requires a passing TGP before merging is allowed and removed target: minor This PR is targeted for the next minor release action: review The PR is still awaiting reviews from at least one requested reviewer detected: feature PR contains a feature commit labels Dec 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accessibility This issue is related to accessibility (a11y) action: global presubmit The PR is in need of a google3 global presubmit area: material/slide-toggle dev-app preview When applied, previews of the dev-app are deployed to Firebase docs: preview When applied, a preview of the documentation site is deployed to Firebase requires: TGP This PR requires a passing TGP before merging is allowed target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants