[Webkit-unassigned] [Bug 21794] Middle-click panning should be springloaded while dragging

[bugzilla-daemon at webkit.org]

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





------- Comment #37 from pkasting at google.com  2009-06-01 09:59 PDT -------
(In reply to comment #35)
> The title of this one is "Middle-click panning
> should be springloaded
> while dragging" which is implemented by this patch as a WebKit option.

No, according to you (and my reading of the patch code), this implements
"Middle-click panning should be springloaded always" as an option.

> > If this is in fact what it does, no client wants this.  Certainly what Chromium
> > wants (and what I suspect Safari will want) is more what Loren's patch aimed
> > for: spring-loaded autoscroll on drag, sticky otherwise.
> That is wrong. See below.

Your pasted motivation below is a user comment on a Chromium bug.  This comment
implies nothing about what Chromium as a WebKit client is actually going to do.
 What Chromium wants, as explicitly specified by the UI design team, is to
springload autoscroll on middle-click drag, and to have sticky autoscroll on
middle click, with no options.  This is also exactly what this bug requests,
and what Loren's original patch was aiming for.

Furthermore, even if the comment was somehow representative of Chromium's
design position, it requests the ability to toggle off autoscrolling entirely,
a la bug 24791, not the option you have implemented here.

I am now convinced that the patch as landed is undesirable to everyone and
should be backed out entirely.  It never belonged on this bug to begin with.

Itai, next time you want to "fix" some Chromium bugs upstream, clear your
solutions with the Chromium UI team *before* implementing them, not after or,
in this case, never.  See the email thread Ben Goodger started recently on
chromium-dev at .


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