-
Notifications
You must be signed in to change notification settings - Fork 34
Remove CKEditor instance on unmount #29
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
base: master
Are you sure you want to change the base?
Conversation
Fixed an issue where new or changed event handlers wouldn't register on existing component. |
… the editor instance is created
Any update on this. Seeing this issue. |
@hafeezpk the PR has never been merged, you can try installing my fork from npm if you want to, the package name is r0nd_react-ckeditor-component |
@hafeezpk could you post some code? Doesn't reproduce with a trivial case. |
I'm just rendering multiple instances of CKEditor. Although the point to note is, manually on the web browser we never see this issue. Using karma (chrome) which automatically launches web browser for functional testing, we see the issue. |
Seems to fix the sometimes appearing "Uncaught TypeError: Cannot call method 'unselectable' of null" errors