Summary: | ARIA spinbutton role incorrectly mapped to ProgressIndicatorRole | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Steve Faulkner <faulkner.steve> | ||||
Component: | Accessibility | Assignee: | chris fleizach <cfleizach> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | bdakin, cfleizach, dmazzoni, faulkner.steve, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Steve Faulkner
2012-01-29 07:58:01 PST
I believe changing the current line http://trac.webkit.org/browser/trunk/Source/WebCore/accessibility/AccessibilityObject.cpp#L1402 from { "spinbutton", ProgressIndicatorRole }, to { "spinbutton", IncrementorRole }, would resolve the issue there may be some changes required in other files for other platforms as in Windows the appropriate mapping would likely be { "spinbutton", SpinButtonRole }, Examples of javascript widgets affected by this bug: in the Ext GWT library the simple form the spin button is identified using ARIA role=spinbutton , it is incorrectly exposed as progress indicator http://dev.sencha.com/playpen/gxt/aria2/test.html?id=focusmanager in the Dojo digit library http://archive.dojotoolkit.org/nightly/checkout/dijit/tests/form/test_Spinner.html Created attachment 151258 [details]
patch
Comment on attachment 151258 [details] patch Clearing flags on attachment: 151258 Committed r122372: <http://trac.webkit.org/changeset/122372> All reviewed patches have been landed. Closing bug. |