-
Notifications
You must be signed in to change notification settings - Fork 370
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
Changes made using the editor buttons doesn't trigger onchange on the textarea #308
Comments
I had the same problem. A change may not be captured because a button did not trigger the change event. I found the following in the code:
I'm not very good with jQuery but I think there may be two problems with this:
Given those two issues I change the code to:
Interactive testing seemed to show that this works. |
|
The discussion in refactory-id#308 indicates that `this.change(this)` is incorrect, adding a listener instead of triggering a change event.
@lodev09 the bug that you have confirmed in your comment exists within this repo, it is not something that users of this library can change. Can this issue be re-opened until the fix is merged? |
The discussion in refactory-id#308 indicates that `this.change(this)` is incorrect, adding a listener instead of triggering a change event.
To reproduce follow these steps:
onchange
events for the textarea by executing the following in the browser console. This selects the first textarea in the demo page at the time of writing.$('textarea[name=content]').change(console.log.bind(console))
onchange
event was logged to the console.onchange
event was triggered.An
onchange
event should be triggered in this case.The text was updated successfully, but these errors were encountered: