-
Notifications
You must be signed in to change notification settings - Fork 928
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
Placeholder is not updating #340
Comments
Care to PR in a fix? |
If you put
|
Ok, let's defer to the Quill recommendation for handling placeholder. I'll add a note to the README. |
…er` as a prop if we want it to dynamically update. Using a random number as a key was a _bad_ idea, since it destroyed focus and history for the quill editors every time it changed. (Punching through to the dataset of the root element seems dirty, but it appears to be the suggested way to do this, per the Quill authors.)
…er` as a prop if we want it to dynamically update. Using a random number as a key was a _bad_ idea, since it destroyed focus and history for the quill editors every time it changed. (Punching through to the dataset of the root element seems dirty, but it appears to be the suggested way to do this, per the Quill authors.)
I know this issue is closed since a long time, but I would expect that this should be an internal responsibility of react-quill instead of forwarding this responsibility to the users. You could make the
With the current implementation, your users are responsible for doing this themselves, and the nature of the |
If I try to re-render my component with a different placeholder - the placeholder is not changing.
Looking into: react-quill/lib/component.js
The placeholder is not on
dirtyProps
listSee example:
https://codepen.io/dbabaioff/pen/pLJyLq
React-Quill version
[*] 1.2.0
The text was updated successfully, but these errors were encountered: