-
Notifications
You must be signed in to change notification settings - Fork 218
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
Additional states "events", "caret", "typing", "plaintext-only" #84
Labels
Comments
dinukadesilva
added a commit
to dinukadesilva/react-contenteditable
that referenced
this issue
Mar 31, 2018
dinukadesilva
added a commit
to dinukadesilva/react-contenteditable
that referenced
this issue
Mar 31, 2018
dinukadesilva
added a commit
to dinukadesilva/react-contenteditable
that referenced
this issue
Mar 31, 2018
good PR, but it's much cleaner to put all of these elements in a the single array on L46.. You can avoid all if statements this way |
By the way, events like |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
https://w3c.github.io/editing/contentEditable.html#h-contenteditable
According to the current implementation, how about enabling these via props. For now, only the disabled prop is there to toggle between "true" and "false" states. So, I was just wondering how about having the other states too as props
Any thoughts?
The text was updated successfully, but these errors were encountered: