Opened 10 years ago
Last modified 10 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 10 years ago by
| Keywords: | Blink added; widget removed |
|---|---|
| Status: | new → confirmed |
| Version: | 4.5.8 → 4.3 |
comment:2 Changed 10 years ago by
| Owner: | set to Tade0 |
|---|---|
| Status: | confirmed → assigned |
comment:3 Changed 10 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 10 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 10 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.
h2todivand 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.