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

Remove CKEditor instance on unmount #29

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

R0nd
Copy link
Contributor

@R0nd R0nd commented Mar 21, 2018

Seems to fix the sometimes appearing "Uncaught TypeError: Cannot call method 'unselectable' of null" errors

@R0nd
Copy link
Contributor Author

R0nd commented Mar 22, 2018

Fixed an issue where new or changed event handlers wouldn't register on existing component.

@hafeezpk
Copy link

hafeezpk commented Aug 7, 2018

Any update on this. Seeing this issue.

@R0nd
Copy link
Contributor Author

R0nd commented Aug 7, 2018

@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
Copy link

hafeezpk commented Aug 7, 2018

R0nd i did try the package r0nd_react-ckeditor-component. Still seeing the same issue.
screen shot 2018-08-07 at 9 01 16 pm

@R0nd
Copy link
Contributor Author

R0nd commented Aug 7, 2018

@hafeezpk could you post some code? Doesn't reproduce with a trivial case.

@hafeezpk
Copy link

hafeezpk commented Aug 8, 2018

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants