[Webkit-unassigned] [Bug 115517] Selection in non-editable content gets stuck when approaching an editable node

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=115517





--- Comment #6 from Claudio Saavedra <csaavedra at igalia.com>  2013-05-06 12:07:03 PST ---
(From update of attachment 200415)
View in context: https://bugs.webkit.org/attachment.cgi?id=200415&action=review

>> Source/WebCore/editing/VisiblePosition.cpp:458
>> +    // FIXME: We skip non-editable regions inside the lowestEditableAncestor of pos for simplicity.
> 
> I don’t understand this FIXME. It seems like this is just stating the current behavior.
> FIXME should mention what needs to be fixed.

Current behavior is to stop and return a null VisiblePosition. The new behavior is to move to the next editable region *but* we skip non-editable regions that might be inside the lowestEditableAncestor, like for instance in "<span editablecontent><span id="non-editable" editablecontent=false>foo</span></span><span id="next-editable" contenteditable>bar</span>", we skip "non-editable" and just move to "next-editable". What needs to be done to remove the new FIXME, is to traverse lowestEditableAncestor() and find the first non-editable that's a descendant of it, before skipping to its next/previous sibling.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.