Uh oh!
There was an error while loading. Please reload this page.
fix(text-field): autosize textarea not resizing on minRows decrease - #13437
Conversation
| get minRows(): number { return this._minRows; } | ||
| set minRows(value: number) { | ||
| if (value !== this._minRows) { | ||
| this._minRowsChanged = true; |
There was a problem hiding this comment.
Should be able to reduce this to this._minRowsChanged = value !== this._minRows. It's unlikely for this to be called quickly enough as to have the _minRowsChanged be overwritten before we've had a chance to resize.
There was a problem hiding this comment.
I think even better is to keep track of the number at the last resize rather than a boolean. That way if they change it to something new and then back to the original we can skip the work (I don't know if this will ever come up in real situations, but technically more correct)
6341079 to
be82d57Compare
This comment has been minimized.
This comment has been minimized.
1 similar comment
Hi @jelbourn! This PR has merge conflicts due to recent upstream merges. |
be82d57 to
d9c4d14Compare
This comment has been minimized.
This comment has been minimized.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Fixes#13163