[Webkit-unassigned] [Bug 71128] Select multiple options with mouse drag in Select element.

[bugzilla-daemon at webkit.org]

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





--- Comment #15 from Kent Tamura <tkent at chromium.org>  2011-11-02 01:22:06 PST ---
(In reply to comment #4)
> (In reply to comment #3)
> > What is the difference with bug 70496?
> > 
> This change will not have effect on this 70496 as "canAutoscroll" fails for the select element with size more than the option elements which is the case for "Priority" in the link given for bug. (Sorry, the description of that issue is not clear and will update it accordingly)

I'm  still confused with this patch and Bug 70496. The test in your patch looks to confirm Bug 70496 is fixed. Please clarify the description of this bug and the patch.

(In reply to comment #10)
> (In reply to comment #9)
> > Presently we return if don't find a renderer for targetNode in EventHandler::handleMouseDraggedEvent() :
> > 
> > if (event.event().button() != LeftButton || !targetNode || !targetNode->renderer())
> > 
> > When we do a drag on the option element we don't get a renderer but as its parent(select) element has a renderer and it can scroll, we can allow auto scroll for parent.
> 
> Then we must be detecting this specific case, not always fallback to the parent node.

I agree with Ryosuke.  We should have isListBox() check.

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