Skip to content

fix(material/button-toggle): use smaller border radius #31084

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

Merged
merged 1 commit into from
May 12, 2025

Conversation

crisbeto
Copy link
Member

Currently the button toggle uses the corner-full value for its broder radius which starts too look weird for a vertical group with text inside the toggles.

These changes switch to using the corner-extra-large value instead.

Fixes #31010.

Currently the button toggle uses the `corner-full` value for its broder radius which starts too look weird for a vertical group with text inside the toggles.

These changes switch to using the `corner-extra-large` value instead.

Fixes angular#31010.
@crisbeto crisbeto requested a review from a team as a code owner May 12, 2025 07:17
@crisbeto crisbeto added the target: patch This PR is targeted for the next patch release label May 12, 2025
@crisbeto crisbeto requested review from mmalerba and wagnermaciel and removed request for a team May 12, 2025 07:17
@andrewseguin andrewseguin added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label May 12, 2025
Copy link

Deployed dev-app for 13603e1 to: https://ng-dev-previews-comp--pr-angular-components-31084-dev-pkgqrxgp.web.app

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

@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label May 12, 2025
@crisbeto crisbeto merged commit 90413f9 into angular:main May 12, 2025
29 of 32 checks passed
@crisbeto
Copy link
Member Author

The changes were merged into the following branches: main, 19.2.x, 20.0.x

crisbeto added a commit that referenced this pull request May 12, 2025
Currently the button toggle uses the `corner-full` value for its broder radius which starts too look weird for a vertical group with text inside the toggles.

These changes switch to using the `corner-extra-large` value instead.

Fixes #31010.

(cherry picked from commit 90413f9)
crisbeto added a commit that referenced this pull request May 12, 2025
Currently the button toggle uses the `corner-full` value for its broder radius which starts too look weird for a vertical group with text inside the toggles.

These changes switch to using the `corner-extra-large` value instead.

Fixes #31010.

(cherry picked from commit 90413f9)
wildcardalice pushed a commit to wildcardalice/components that referenced this pull request May 12, 2025
Currently the button toggle uses the `corner-full` value for its broder radius which starts too look weird for a vertical group with text inside the toggles.

These changes switch to using the `corner-extra-large` value instead.

Fixes angular#31010.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: material/button-toggle dev-app preview When applied, previews of the dev-app are deployed to Firebase target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(COMPONENT): Vertical mat-button-toggle layout is too round and truncate content
2 participants