Skip to content

fix(material/form-field): position outlined form field label correctl… #27525

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 1 commit into
base: main
Choose a base branch
from

Conversation

behzadmehrabi
Copy link
Contributor

@behzadmehrabi behzadmehrabi commented Jul 27, 2023

Fixes issues with the outlined form-field label offset not updating correctly when the form-field was attached to the DOM but not visible, or when the prefix container had dynamic content.

Fixes #26332

@behzadmehrabi behzadmehrabi force-pushed the fix-outlined-form-field-label-offset branch 2 times, most recently from 932f0a7 to 3308bed Compare July 28, 2023 20:31
@behzadmehrabi behzadmehrabi requested a review from mmalerba July 28, 2023 20:40
@behzadmehrabi behzadmehrabi force-pushed the fix-outlined-form-field-label-offset branch 2 times, most recently from a9fdd9a to d4b8e13 Compare July 29, 2023 20:51
@behzadmehrabi behzadmehrabi force-pushed the fix-outlined-form-field-label-offset branch 3 times, most recently from 37b4f2a to f112392 Compare August 2, 2023 00:14
…nd prefix content change

Fixes issues with the outlined form-field label offset not updating correctly when the form-field was attached to the DOM but not visible, or when the prefix container had dynamic content.
@behzadmehrabi behzadmehrabi force-pushed the fix-outlined-form-field-label-offset branch from f112392 to 98edad0 Compare August 2, 2023 00:44
@behzadmehrabi behzadmehrabi requested a review from mmalerba August 3, 2023 07:57
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

One last thing, then it looks good

@devversion devversion removed their request for review August 7, 2023 12:42
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

LGTM

@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 Aug 8, 2023
@mmalerba mmalerba self-assigned this Aug 16, 2023
@mmalerba
Copy link
Contributor

Hi @behzadmehrabi, I tried running the internal tests for this and got a lot of failures because the tests were throwing ResizeObserver loop limit exceeded errors. We need to investigate why these are happening and convince ourselves that it is not a performance issue. (I had to go through a similar exercise when adding the label resizing support.) Once we confirm that the errors are benign, we'll need a similar setTimeout workaround so the errors don't break tests

@mmalerba mmalerba added presubmit failures This PR has failures in Google's internal presubmit process and cannot be immediately merged and removed action: merge The PR is ready for merge by the caretaker labels Aug 17, 2023
@mmalerba mmalerba removed their assignment Sep 25, 2023
@josephperrott josephperrott requested a review from a team as a code owner December 18, 2024 17:40
@josephperrott josephperrott requested review from crisbeto and amysorto and removed request for a team December 18, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit failures This PR has failures in Google's internal presubmit process and cannot be immediately merged 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): Label position is wrong (appearance="outline") when mat-form-field is placed inside drawer
2 participants