-
Notifications
You must be signed in to change notification settings - Fork 673
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
Use min-height of textarea when computing auto-resize after deleting a char #504
Conversation
@@ -1260,6 +1260,7 @@ button { | |||
} | |||
|
|||
#form #input { | |||
overflow-y: hidden; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will break scrolling when it hits 6-7 lines, as we stop growing at this point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, good point, I'll remove this.
Functionality-wise works fine, 👍 (except the overflow issue) |
332ee84
to
6abb9f1
Compare
6abb9f1
to
cc06081
Compare
cc06081
to
27b33f4
Compare
@maxpoulin64 is 👍 (but didn't comment with 👍 itself, just the second-review label). All other maintainers are away at the moment, but @williamboman did test this successfully, so merging. |
…xtarea-height Use min-height of textarea when computing auto-resize after deleting a char
Fixes #482.
This fixes a bug that was breaking the textarea on Android.
@williamboman, can you give it a try?