-
-
Notifications
You must be signed in to change notification settings - Fork 397
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(contentarea): hide multiline text in contentarea #1189
fix(contentarea): hide multiline text in contentarea #1189
Conversation
fixed hidden multiline content taking up display space on hover
use reduced dimensions to hide contentarea instead of display: none; for improved performance
f5ba241
to
45daba9
Compare
@shanberg review? explanation at #1131 (comment) Seems to fix the ghost hover effect, but then there is a flicker when blurring from the textarea: https://www.loom.com/share/8358b3c91f004b1ebca2c20574fbf3ad?focus_title=1&muted=1&from_recorder=1 |
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.
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.
Clicking should place caret in approximately the right location.
remove flicker during textarea blur
6bbf6a4
to
563fb7e
Compare
caret position in textarea fixed. Also fixed the flicker on textarea blur |
I'm impressed with your fixes here @ozimos. I'd sort of assumed we'd hit a local maximum with the textarea-based UX, but clearly it can still improve. Demo gifts attached. It’s maddening to see both animations together so I've linked them instead. |
…1189) * fix: improve css for textarea when not editing fixed hidden multiline content taking up display space on hover * fix: remove display none on contentarea use reduced dimensions to hide contentarea instead of display: none; for improved performance * fix: clicking inserts caret in textarea remove flicker during textarea blur Co-authored-by: jeff <[email protected]>
close #1131 by setting the default line-height on contentarea to 0.
close #613 (duplicate)