Skip to content
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

Clicking on editable placeholder should hide text #806

Closed
mtias opened this issue May 16, 2017 · 3 comments
Closed

Clicking on editable placeholder should hide text #806

mtias opened this issue May 16, 2017 · 3 comments
Assignees
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Type] Enhancement A suggestion for improvement.

Comments

@mtias
Copy link
Member

mtias commented May 16, 2017

When clicking on editable's placeholder we should probably remove the placeholder text:

image

@mtias mtias added [Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Type] Enhancement A suggestion for improvement. labels May 16, 2017
@ellatrix
Copy link
Member

At the moment it's the same behaviour as a normal input field, which also leaves the text there on focus: https://codepen.io/iseulde/pen/mmjRYV

@jasmussen
Copy link
Contributor

I tend to agree with Ella on this one. It feels a little weird since the text is centered here, but it's actually sort of how it "should" work.

@mtias
Copy link
Member Author

mtias commented May 16, 2017

Maybe the cursor and text alignment given the centered position is throwing me off.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Rich Text Related to the Rich Text component that allows developers to render a contenteditable [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

No branches or pull requests

3 participants