Opened 9 years ago
Last modified 8 years ago
#13778 review Bug
[Chrome] Trying to insert a block before a block widget fails with an error
Reported by: | Piotrek Koszuliński | Owned by: | Tade0 |
---|---|---|---|
Priority: | Nice to have (we want to work on it) | Milestone: | |
Component: | General | Version: | 4.3 |
Keywords: | Cc: |
Description
Steps to reproduce
- http://jsfiddle.net/HugeHugh/3xu8yece/
- Try to insert the horizontal line without focusing the editor first.
Expected result
It's inserted before the widget.
Actual result
An error is thrown.
Other details (browser, OS, CKEditor version, installed plugins)
I could not reproduce this issue on Firefox, so I guess it's related to Chrome's limited selection system which does not allow us to place selection before a non-editable block if it's the first descendant of the editable.
Attachments (1)
Change History (15)
comment:1 Changed 9 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 9 years ago by
Changed 9 years ago by
Attachment: | mytest.html added |
---|
comment:4 Changed 9 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 9 years ago by
Status: | assigned → review |
---|
The previous method of ensuring that a range to which the element should've been inserted existed was not robust enough. Fixed that by creating a range when none is available.
Changes pushed to branch:t/13778.
comment:6 Changed 9 years ago by
Milestone: | CKEditor 4.5.6 → CKEditor 4.5.7 |
---|
comment:7 Changed 9 years ago by
Greetings. I'm unsure how to test the code change. When I go to the referenced branch on Github (t/13778) I see there was a change made. However, I use a pre-packaged build of CKEditor that was created by your website, and I'm not sure how to bring this change into my build of CKEditor that I configured on your website.
Here's the direct link to my build config: http://ckeditor.com/builder/544a1d22c6dfe8e23af36b1eee82633d
I just need some help to get the bug fix from this ticket into my build so that our app will begin functioning correctly. We've been waiting for this to be fixed since I first reported it 9/16/2015. Thanks so much for your help!
comment:8 Changed 9 years ago by
Milestone: | CKEditor 4.5.7 → CKEditor 4.5.8 |
---|
comment:9 Changed 9 years ago by
Milestone: | CKEditor 4.5.8 → CKEditor 4.5.9 |
---|
comment:10 Changed 9 years ago by
Milestone: | CKEditor 4.5.9 → CKEditor 4.5.10 |
---|
comment:11 Changed 8 years ago by
Milestone: | CKEditor 4.5.10 → CKEditor 4.5.11 |
---|
Moving tickets to the next milestone.
comment:12 Changed 8 years ago by
Milestone: | CKEditor 4.5.11 → CKEditor 4.6.1 |
---|
comment:13 Changed 8 years ago by
Milestone: | CKEditor 4.6.1 → CKEditor 4.6.2 |
---|
Moving to 4.6.2 minor release, as 4.6.1 is mostly about polishing 4.6.0.
comment:14 Changed 8 years ago by
Milestone: | CKEditor 4.6.2 |
---|---|
Priority: | Normal → Nice to have (we want to work on it) |
Moving to the nice to have list.
Inquire about status of this ticket. It is important to me.
Discussion in related ticket is here: http://dev.ckeditor.com/ticket/13749
Your help is greatly appreciated.