Skip to content
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

fix(material/form-field): account in cols attribute on textarea #29836

Merged
merged 1 commit into from
Oct 10, 2024

Conversation

naaajii
Copy link
Contributor

@naaajii naaajii commented Oct 6, 2024

previously we were setting width of 180px on our input field however this also made that textarea width stays the same if they are used with cols attribute, this commit ensures we dont do that if we have a textarea with cols attribute allowing it to grow in width as much as the attribute wants it to

fixes #29459

@naaajii naaajii requested a review from a team as a code owner October 6, 2024 15:28
@naaajii naaajii requested review from crisbeto and andrewseguin and removed request for a team October 6, 2024 15:28
previously we were setting width of 180px on our input field however this also made that textarea width stays the same if they are used with `cols` attribute, this commit ensures we dont do that if we have a textarea with `cols` attribute allowing it to grow in width as much as the attribute wants it to

fixes angular#29459
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Oct 10, 2024
@crisbeto crisbeto removed the request for review from andrewseguin October 10, 2024 16:44
@crisbeto crisbeto merged commit 48b2680 into angular:main Oct 10, 2024
26 checks passed
crisbeto pushed a commit that referenced this pull request Oct 10, 2024
…9836)

previously we were setting width of 180px on our input field however this also made that textarea width stays the same if they are used with `cols` attribute, this commit ensures we dont do that if we have a textarea with `cols` attribute allowing it to grow in width as much as the attribute wants it to

fixes #29459

(cherry picked from commit 48b2680)
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 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): 'cols' attribute of <textarea> does not work with mat-form-field
2 participants