Skip to content

Conversation

mmalerba
Copy link
Contributor

@mmalerba mmalerba commented Aug 30, 2024

Fixes #26558

Note: the label may still briefly be larger than the form-field, during transition from resting to floating. Users may choose to add a CSS animation for this, but we don't want to bake it in because it may negatively impact performance.

@mmalerba mmalerba requested a review from a team as a code owner August 30, 2024 22:55
@mmalerba mmalerba requested review from andrewseguin and crisbeto and removed request for a team August 30, 2024 22:55
@mmalerba mmalerba added the G This is is related to a Google internal issue label Aug 31, 2024
padding-left: 16px;
padding-right: 16px;

@include _supports-max {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heads-up that we inherited this _supports-max logic from MDC and reducing the specificity might be break-ey during the presubmit.

@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Sep 9, 2024
@mmalerba mmalerba removed the request for review from andrewseguin September 9, 2024 20:22
@mmalerba mmalerba merged commit 77051f8 into angular:main Sep 9, 2024
26 checks passed
mmalerba added a commit that referenced this pull request Sep 9, 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 Oct 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker G This is is related to a Google internal issue 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(mat-form-field): Form field with icon prefix or suffix makes long label overflow

2 participants