Bug 18167 – there should be no text-drag delay on any platform except for Mac

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Tony Chang]

TextDragDelay is the time between mouse down and mouse move that needs to pass before selected text can be dragged.  On windows and linux, it seems like this time should be zero.

I have an upcoming patch that makes this value platform specific.

[Tony Chang]

Created attachment 20139 [details]
make TextDragDelay platform specific

Make TextDragDelay platform specific and 0 on non-mac platforms.

[Eric Seidel (no email)]

Patch looks good to me.  aroben or sfalken should review though.

[Darin Adler]

Comment on attachment 20139 [details]
make TextDragDelay platform specific

This doesn't just make this platform-specific; it sets it to 0 for almost every platform.

What's the argument for having no delay on Windows and all the Windows-copycat platforms? A value of 0 seems wrong for any platform! Am I missing something?

[Darin Adler]

Retitling the bug to match what the patch actually does. Is this really what we want to do?

[Tony Chang]

I can't see any evidence of there being a delay on Windows.  The Vista UI guidelines make no mention of a delay:
http://msdn2.microsoft.com/en-us/library/aa511258.aspx (page 384 of the pdf)

Empirical evidence from using Windows software and Firefox's source code suggest there's no delay before a drag starts (this is also true of Firefox on mac, but that seems to be a bug).
http://mxr.mozilla.org/seamonkey/source/content/events/src/nsEventStateManager.cpp#848

[Eric Seidel (no email)]

I agree.  no-delay seems wrong to me.  But it also definitely seems to be what Windows does. :(  aroben or sfalken should really chime in here.

[Eric Seidel (no email)]

Comment on attachment 20139 [details]
make TextDragDelay platform specific

I asked Steve Falkenburg over email, he agrees, this delay-less behavior is correct.  r+

[Darin Adler]

Committed revision 34449.