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

Cannot read property 'unselectable' of null #114

Closed
boris27 opened this issue Jul 3, 2020 · 13 comments
Closed

Cannot read property 'unselectable' of null #114

boris27 opened this issue Jul 3, 2020 · 13 comments
Labels
status:blocked An issue which development is blocked by another issue (internal or external one). status:confirmed An issue confirmed by the development team. type:bug A bug.

Comments

@boris27
Copy link

boris27 commented Jul 3, 2020

Are you reporting a feature request or a bug?

Bug

hi guys, i have little different situation(which described in #110 ), and your fix in 1.2.2 doesnt work for me

this is example of above code, little modified to exampled my situation)
https://stackblitz.com/edit/angular-ivy-taveju?file=src%2Fapp%2Fapp.component.ts

@f1ames
Copy link
Contributor

f1ames commented Jul 6, 2020

@boris27 could you elaborate a little bit more on your real use case? Do you use such code in your Angular application or it is just here to somehow simulate real use case? Isn't it related to an Angular issue described here - #111 (comment) 🤔

@jacekbogdanski
Copy link
Member

Hi, It's been a while since we last heard from you. We're closing this issue for now. Still, feel free provide us requested feedback, so that we can reopen this issue.

@jacekbogdanski jacekbogdanski added resolution:expired Issue reporter did not provide enough information to reproduce the issue for at least 2 weeks. and removed status:pending labels Jul 20, 2020
@s-abdullaev
Copy link

s-abdullaev commented Jul 24, 2020

Same issue is happening in my project as well. It is reporting following error:

Uncaught TypeError: Cannot read property 'unselectable' of null
    at e (ckeditor.js:349)
    at a.<anonymous> (ckeditor.js:345)
    at a.d (ckeditor.js:10)
    at a.<anonymous> (ckeditor.js:11)
    at a.CKEDITOR.editor.CKEDITOR.editor.fire (ckeditor.js:13)
    at a.fireOnce (ckeditor.js:12)
    at a.CKEDITOR.editor.CKEDITOR.editor.fireOnce (ckeditor.js:13)
    at Object.<anonymous> (ckeditor.js:272)
    at g (ckeditor.js:252)
    at Object.load (ckeditor.js:252)

For simple use case inside a pop-up dialog. For example,:

<p-dialog>
<ckeditor
                #editor
                formControlName="body"
                [config]="ckconfig"
></ckeditor>
</p-dialog>

@f1ames
Copy link
Contributor

f1ames commented Jul 27, 2020

@desmonduz could you provide a demo (e.g. on stackblitz or similar) where your issue could be reproduced? I wonder if it's caused by placing editor inside dialog/modal (p-dialog).

@s-abdullaev
Copy link

Hi @f1ames

I have made a simple demo here:
https://stackblitz.com/edit/angular-ivy-grhjxj

@xxj2016
Copy link

xxj2016 commented Mar 4, 2021

Same issue is happening in my project as well.it may be caused by placing editor inside modal

@ronaldohoch
Copy link

Same issue here.

In my case: When i use ckeditor inside Angular Material Tabs, the first tab work properly, but, the second tab, with the hidden content, throws this error.
And, when i go to second tab and come back to the first tab, the ckeditor is not working anymore.

@Dumluregn
Copy link
Collaborator

I'm reopening this issue since the problem is still there. For everyone interested - it's an upstream from ckeditor4 itself, we created a generic issue there (ckeditor/ckeditor4#4390) with two follow-ups (ckeditor/ckeditor4#4461 and ckeditor/ckeditor4#4462) we are actively working on (one is already merged to the major branch, the other comes through the second review). After the release containing those two PRs (probably 4.17.0) we'll update the configuration in sample and all UI frameworks should start working fine.

@Dumluregn Dumluregn reopened this May 13, 2021
@Dumluregn Dumluregn added size:External status:blocked An issue which development is blocked by another issue (internal or external one). status:confirmed An issue confirmed by the development team. type:bug A bug. and removed resolution:expired Issue reporter did not provide enough information to reproduce the issue for at least 2 weeks. labels May 13, 2021
@Dumluregn
Copy link
Collaborator

Closing #190 should be at least a partial fix.

@Dumluregn
Copy link
Collaborator

There is another nice debugging sample provided by a user in #192 (closed as duplicate).

@Dumluregn
Copy link
Collaborator

This issue should be resolved with #190.

@sculpt0r
Copy link
Contributor

Hi,
The background laying issue was fixed in CKEditor v4.17.

  • The angular integration 2.3.0 is by default linked to CKE4 4.17.1, but it requires to use of proper config.
  • The 2.4.0 version introduces (in this integration) the default usage of the delayed editor initialization feature (blog post).
  • Currently, the newest version is 2.4.1 - see notes.

Since the issue was fixed. I'm closing this report. If you encourage any further issues - feel free to open another issue. Thank you for your patience.

@sculpt0r
Copy link
Contributor

Closed with #190.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:blocked An issue which development is blocked by another issue (internal or external one). status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Development

No branches or pull requests

8 participants