Bug 44476

Summary: <input type=number>: Support auto-repeat by mouse press
Product: WebKit Reporter: Kent Tamura <tkent>
Component: FormsAssignee: Kent Tamura <tkent>
Status: RESOLVED FIXED    
Severity: Normal CC: adele, dglazkov, fishd, hamaji
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 27968    
Attachments:
Description Flags
Patch
none
Patch 2 none

Description Kent Tamura 2010-08-23 18:25:39 PDT
<input type=number>: Support auto-repeat by mouse press
Comment 1 Kent Tamura 2010-08-23 18:30:44 PDT
Created attachment 65197 [details]
Patch
Comment 2 Shinichiro Hamaji 2010-08-25 01:23:54 PDT
Comment on attachment 65197 [details]
Patch

This change looks good, but I think the behavior isn't the best when

1. mouse down on increase button
2. mouse move from increase button to decrease button without mouse up
3. the number will decrease

It seems with windows' standard GUI, the number doesn't change in step 3. I guess this behavior or increasing will be better than the behavior of this patch. I don't have access to Mac for now, could you check what happens on Mac?
Comment 3 Kent Tamura 2010-08-25 01:36:25 PDT
(In reply to comment #2)
> (From update of attachment 65197 [details])
> This change looks good, but I think the behavior isn't the best when
> 
> 1. mouse down on increase button
> 2. mouse move from increase button to decrease button without mouse up
> 3. the number will decrease
> 
> It seems with windows' standard GUI, the number doesn't change in step 3. I guess this behavior or increasing will be better than the behavior of this patch. I don't have access to Mac for now, could you check what happens on Mac?

I agree with you.  I tested the behavior of Mac (SnowLoepard, System Preference -> Date & Time) and found the Mac behavior was the same as this patch :-(
I'll update the patch.
Comment 4 Kent Tamura 2010-08-25 02:02:01 PDT
Created attachment 65385 [details]
Patch 2
Comment 5 Shinichiro Hamaji 2010-08-25 02:16:34 PDT
Comment on attachment 65385 [details]
Patch 2

Looks good. Thanks for checking the behavior of Mac.
Comment 6 Kent Tamura 2010-08-25 02:46:04 PDT
Landed as r66001.