-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Expand web TextInput as the user types #284
Conversation
Going to be OOO until 8/31, if this needs to be updated feel free to ping somebody else to update. If not, I'll get back to it then! |
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.
LGTM and works well!
Gonna hold off merging this until next week in case it breaks something (doesn't look like it should) |
What's the latest on this one? |
I agree that a max-height for the input would be great. |
Heads up, I just moved the related issue to high. We're already running into this on a daily basis, and is a very important component of moving DMs into RNC. |
This fell off my radar while helping out implement DMs. Merging now so we should be 👍 |
Fixes #103
Tests