Opened 13 years ago
Closed 12 years ago
#8569 closed Bug (fixed)
FF: JS error when inserting elements after HTML with comments.
Reported by: | Jakub Ś | Owned by: | Frederico Caldeira Knabben |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.6.1 |
Keywords: | IE | Cc: |
Description (last modified by )
Ticket has been reported on our support channel - Topic - "Possible bug: HTML comments cause Javascript error when integrating CKFinder with CKEditor"
- Insert on of the following
<!--A comment--> <div><!--A comment--></div> <div>Something else</div><!-- A comment --> <div><div>Something else</div><!-- A comment --></div>
- Try to insert Image/Table/Link/Page break using toolbar buttons.
- After pressing OK JS error pops out.
The error:
Message: startNode.getChildCount is not a function
Line: 792
URI: ckeditor/_source/core/dom/range.js
Bug is only reproducible in Firefox 4-10 and IE 6-9 (FF 3.6 is fine).
Ticket has been reproducible from CKE 3.6.1 rev [6963]
NOTE:
Before this revision another error was thrown (from CKE 3.3)
Message: startContainer.getChild is not a function
URI: /3.6/_source/core/dom/range.js
Line: 1472
Attachments (1)
Change History (10)
comment:1 Changed 13 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 13 years ago by
Description: | modified (diff) |
---|
comment:3 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Keywords: | IE added |
comment:5 Changed 12 years ago by
Status: | confirmed → pending |
---|
I as not able to reproduce this issue on trunk.
There is one thing that is not clear for me in the tc... where should the selection be before trying to insert?
comment:6 Changed 12 years ago by
Keywords: | Firefox removed |
---|---|
Status: | pending → confirmed |
I can still reproduce it on trunk.
- Paste in source mode
<!--A comment-->
- switch to wysiwyg
- Click behind comment (In my case when I do click cursor is in the same line as comment but far to the right - below background color button)
- insert table
Reproducible only in IE.
Changed 12 years ago by
Attachment: | 8569.patch added |
---|
comment:7 Changed 12 years ago by
Owner: | set to Frederico Caldeira Knabben |
---|---|
Status: | confirmed → review |
#8812 must be closed first, because both fixes are required to have this ticket fixed.
comment:8 Changed 12 years ago by
Status: | review → assigned |
---|
This one is supposed to get fixed by #8812.
comment:9 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Working on a current trunk.
Similar problem: #8732
Other related to problems with HTML comments: #8640, #8216, #8129, #6709, #8812