Opened 8 years ago
Closed 8 years ago
#16600 closed Bug (fixed)
Editor may be assuming presence of editable too optimistically.
Reported by: | Jakub Ś | Owned by: | Marek Lewandowski |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.6.1 |
Component: | General | Version: | |
Keywords: | Webkit Blink Support | Cc: |
Description (last modified by )
Steps to reproduce
Sometimes in Blink and Webkit browsers when dealing with multiple editors you may encounter a situation where editable is not present yet.
URI: core/dom/element.js
Line: 1986
editable.contains( this ) ) { |
Expected result
No error should be thrown.
Actual result
In these rare cases error might get thrown.
Other details (browser, OS, CKEditor version, installed plugins)
Problem occurs in Blink and Webkit browsers and simple null check should do the trick.
Change History (6)
comment:1 Changed 8 years ago by
Keywords: | Support added |
---|---|
Status: | new → confirmed |
comment:2 Changed 8 years ago by
comment:3 Changed 8 years ago by
Description: | modified (diff) |
---|
comment:4 Changed 8 years ago by
Owner: | set to Marek Lewandowski |
---|---|
Status: | confirmed → review |
Pushed a fix to branch:t/16600.
comment:6 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Merged to master
with a523444.
cc