Bug 127451 - AX: Can't always increment web sliders.
Summary: AX: Can't always increment web sliders.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Accessibility (show other bugs)
Version: 528+ (Nightly build)
Hardware: Macintosh Intel OS X 10.9
: P2 Normal
Assignee: Samuel White
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2014-01-22 15:34 PST by Samuel White
Modified: 2014-01-22 20:08 PST (History)
9 users (show)

See Also:


Attachments
Initial patch for feedback. (955 bytes, patch)
2014-01-22 15:42 PST, Samuel White
no flags Details | Formatted Diff | Diff
Patch. (5.40 KB, patch)
2014-01-22 17:55 PST, Samuel White
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Samuel White 2014-01-22 15:34:48 PST
If a slider has no step size and a very small range, for example:

min = 0, max = 10

AXIncrement and AXDecrement fail. This is because we try and increment by a percent in this case, and the percent results in a value of less than 1. Since 1 is the implicit step size in this case we need to adjust.
Comment 1 Samuel White 2014-01-22 15:35:10 PST
<rdar://problem/15336081>
Comment 2 Samuel White 2014-01-22 15:42:48 PST
Created attachment 221917 [details]
Initial patch for feedback.

Added some basic logic to make sure we step by at least one. I'm doing this to match the keyboard behavior.
Comment 3 chris fleizach 2014-01-22 15:52:46 PST
Comment on attachment 221917 [details]
Initial patch for feedback.

is this the same thing that happens in the keyboard handler. do you have a layout test ready
Comment 4 Samuel White 2014-01-22 17:55:50 PST
Created attachment 221929 [details]
Patch.

Added layout test and updated logs.
Comment 5 Samuel White 2014-01-22 17:56:36 PST
(In reply to comment #3)
> (From update of attachment 221917 [details])
> is this the same thing that happens in the keyboard handler. do you have a layout test ready

Keyboard code also clamps to one in this situation. Layout test has been added.
Comment 6 WebKit Commit Bot 2014-01-22 20:08:28 PST
Comment on attachment 221929 [details]
Patch.

Clearing flags on attachment: 221929

Committed r162587: <http://trac.webkit.org/changeset/162587>
Comment 7 WebKit Commit Bot 2014-01-22 20:08:31 PST
All reviewed patches have been landed.  Closing bug.