Summary: | VO doesn't work with HTML5 range (slider) input @step attribute | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | chris fleizach <cfleizach> | ||||
Component: | Accessibility | Assignee: | chris fleizach <cfleizach> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ddkilzer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
chris fleizach
2011-05-23 12:41:11 PDT
Created attachment 94465 [details]
patch
Comment on attachment 94465 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=94465&action=review r=me, but consider making the test try to go below the minimum value of the slider as well. > Source/WebCore/accessibility/AccessibilityRenderObject.cpp:890 > + changeValueByPercent((increase) ? 5 : -5); Nit: Don't need parenthesis around (increase) here. > Source/WebCore/accessibility/AccessibilityRenderObject.cpp:2224 > + value += (increase) ? step : -step; Nit: Don't need parenthesis around (increase) here. > LayoutTests/platform/mac/accessibility/range-alter-by-step.html:39 > + obj.increment(); > + shouldBe("obj.intValue", "100"); Nice job of testing going over the maximum value! I'd really like to see it try to go below the minimum value as well in this test. |