[Webkit-unassigned] [Bug 60430] selectstart is not fired when selection was created by arrow keys

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Sep 7 09:15:04 PDT 2011


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





--- Comment #6 from Ryosuke Niwa <rniwa at webkit.org>  2011-09-07 09:15:04 PST ---
(In reply to comment #5)
> > This is too late. m_selection.m_isDirectional has already been modified at 
> > this point. r- because of this. You should probably do this in the if- 
> > statement at the beginning of this function.
> 
> We can do this at the beginning of this function. But in IMHO if we keep this in the if-statement at the beginning, selectStart might not execute if modify() function called by JS.

I thought selectstart doesn't fire for programatically set selection anyway. I don't think we currently fire selectstart when a collapsed selection is replaced by a range selection for example, and this behavior matches that of MSIE. You should definitely check MSIE's behavior though. 

> > Why do we need all this code?
> 
> Here we need to find out selectStart target if the element is content editable, Similar as in case of selectAll(), how the selectStart event is fired.

We don't do this elsewhere for firing selectstart so I'd rather not add such a code for this specific case. If this is a bug, then we should fix all those places where we fire selectstart at once.

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



More information about the webkit-unassigned mailing list