Opened 9 years ago
Last modified 9 years ago
#14562 review Bug
triple click and delete ruins widget in chrome
Reported by: | bwaindwain | Owned by: | Tade0 |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | UI : Widgets | Version: | 4.3 |
Keywords: | Blink | Cc: |
Description
Steps to reproduce
- Use chrome
- insert widget with two editable regions that are both div
- type some text in first region
- triple click in first region (to select entire block)
- press delete or backspace key
See http://jsfiddle.net/dbz3gqLe/
Expected result
text should be deleted and the widget should still have two editable regions
Actual result
In CKEditor 4.5.8, the entire first region is deleted.
In CKEditor 4.4.1, the text from second region is moved to the first. See http://jsfiddle.net/o34t8zbk/
Other details (browser, OS, CKEditor version, installed plugins)
chrome 49.0.0263
See Also
http://dev.ckeditor.com/ticket/14353 http://dev.ckeditor.com/ticket/14354 http://dev.ckeditor.com/ticket/12677 http://dev.ckeditor.com/ticket/13395
Change History (5)
comment:1 Changed 9 years ago by
Keywords: | Blink added; widget removed |
---|---|
Status: | new → confirmed |
Version: | 4.5.8 → 4.3 |
comment:2 Changed 9 years ago by
Owner: | set to Tade0 |
---|---|
Status: | confirmed → assigned |
comment:3 Changed 9 years ago by
What I've established so far is that on triple-click the created selection starts at the beginning of the text, but seems to end after the second div.
@bwaindwain Thank you for delivering such a high quality bug report.
comment:4 Changed 9 years ago by
Here's some related discussion at chromium.org - no solutions though.
https://bugs.chromium.org/p/chromium/issues/detail?id=511962 https://bugs.chromium.org/p/chromium/issues/detail?id=575136
comment:5 Changed 9 years ago by
Status: | assigned → review |
---|
I devised a solution that works in this particular case(selection crossing editable boundaries on backspace/delete).
Ideally we would want to correct the selection on selection change, but initial trials show, that the appropriate event is not fired(to investigate).
@bwaindwain I took the liberty of creating a manual test based on your jsFiddle example.
Changes pushed to branch:t/14562.
Seems we are not handling triple-click in this special case for Blink.
h2
todiv
and got result described in TC starting from CKEditor 4.4.2. Before that version (starting from 4.3.0), content from editable one is moved to editable two.