Skip to content

fix(material/chips): remove visible overflow for labels #31679

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

adolgachev
Copy link
Contributor

@adolgachev adolgachev commented Aug 12, 2025

Fixes #26584 and b/221946733 by moving the overridden visible overflow for chip labels to the strong focus mixins. This will let chip labels be correctly truncated (instead of overflowing) if strong focus is not applied (as desired by the community) while preserving strong focus indicators.

@adolgachev adolgachev added Accessibility This issue is related to accessibility (a11y) target: minor This PR is targeted for the next minor release area: material/chips dev-app preview When applied, previews of the dev-app are deployed to Firebase action: review The PR is still awaiting reviews from at least one requested reviewer action: global presubmit The PR is in need of a google3 global presubmit requires: TGP This PR requires a passing TGP before merging is allowed docs: preview When applied, a preview of the documentation site is deployed to Firebase labels Aug 21, 2025
Copy link

github-actions bot commented Aug 21, 2025

Deployed dev-app for e4cea72 to: https://ng-dev-previews-comp--pr-angular-components-31679-dev-topbsor0.web.app

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

Copy link

github-actions bot commented Aug 21, 2025

Deployed docs-preview for e4cea72 to: https://ng-dev-previews-comp--pr-angular-components-31679-docs-v79xie68.web.app

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

@adolgachev adolgachev changed the title fix(chips): remove visible overflow for labels fix(material/chips): remove visible overflow for labels Aug 21, 2025
@adolgachev adolgachev marked this pull request as ready for review August 21, 2025 17:50
@adolgachev adolgachev requested a review from a team as a code owner August 21, 2025 17:50
@adolgachev adolgachev requested review from mmalerba, ok7sai and crisbeto and removed request for a team and ok7sai August 21, 2025 17:50
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 action: review The PR is still awaiting reviews from at least one requested reviewer area: material/chips 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: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(chip): Overflow visible on action chips text break chip design and usability
1 participant