Skip to content

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 andrewseguin and crisbeto 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)
@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 Nov 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 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