Opened 7 years ago

Last modified 7 years ago

#9386 closed Bug

[Webkit] Missing case for filler char in collapsed selection — at Version 3

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.6.5
Component: Core : Selection Version: 3.6.5
Keywords: Cc:

Description (last modified by Frederico Caldeira Knabben)

There's still some logic hole in filler char creating detection, which breaks the following TC:

http://ckeditor.t/tt/7912/1.html

Change History (5)

comment:1 Changed 7 years ago by Garry Yao

Owner: set to Garry Yao
Status: newreview

The patch expanded the filler creation check, in considering of hidden element.

Besides two commonly requested traversal utility are proposed in range.js.

Last edited 7 years ago by Garry Yao (previous) (diff)

Changed 7 years ago by Garry Yao

Attachment: 9386.patch added

comment:2 Changed 7 years ago by Garry Yao

The above patch breaks some current TCs.

Considering that it's not easy to enumerate all possible (see code comments) DOM structure where an empty fix is required, so the proposal takes a new approach - rely on range::moveToElementEditStart/End to check whether the filler is needed, and I've opened t/9386@test that cited all known cases for now.

Changed 7 years ago by Garry Yao

Attachment: 9386_2.patch added

comment:3 Changed 7 years ago by Frederico Caldeira Knabben

Description: modified (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy