Bug 53842

Summary: WAI-ARIA @aria-sort not exposed on rowheader and columnheader roles
Product: WebKit Reporter: chris fleizach <cfleizach>
Component: AccessibilityAssignee: chris fleizach <cfleizach>
Status: RESOLVED FIXED    
Severity: Normal CC: jcraig
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Patch darin: review+

Description chris fleizach 2011-02-04 22:02:07 PST
WAI-ARIA @aria-sort not exposed on rowheader and column header roles

Cocoa table headers are exposed as AXSortButtons, and ARIA table headers are cells. However, ARIA row and col headers can contain an aria-sort attribute with the following values for:

AXSortDirection: AXAscendingSortDirection | AXDescendingSortDirection
Comment 1 chris fleizach 2011-02-06 20:56:12 PST
*** Bug 53876 has been marked as a duplicate of this bug. ***
Comment 2 chris fleizach 2011-02-06 21:14:43 PST
Created attachment 81448 [details]
Patch
Comment 3 Darin Adler 2011-02-07 08:15:00 PST
Comment on attachment 81448 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=81448&action=review

> Source/WebCore/accessibility/mac/AccessibilityObjectWrapper.mm:1990
> +        switch (m_object->sortDirection()) {
> +            case SortDirectionAscending:
> +                return NSAccessibilityAscendingSortDirectionValue;
> +            case SortDirectionDescending:
> +                return NSAccessibilityDescendingSortDirectionValue;
> +            default:
> +                return NSAccessibilityUnknownSortDirectionValue;
> +        }

WebKit style doesn't indent the case inside the switch.
Comment 4 chris fleizach 2011-02-07 09:15:23 PST
Comment on attachment 81448 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=81448&action=review

thanks for the review

>> Source/WebCore/accessibility/mac/AccessibilityObjectWrapper.mm:1990
>> +        }
> 
> WebKit style doesn't indent the case inside the switch.

strange that the style bot didn't catch this one
Comment 5 chris fleizach 2011-02-07 09:24:34 PST
http://trac.webkit.org/changeset/77820
Comment 6 James Craig 2013-07-09 09:49:22 PDT
See also: 
Bug 118511: AX: WebKit doesn't map aria-sort=other