Opened 7 years ago

Closed 7 years ago

#15257 closed defect (fixed)

[Patch][CLA] Dijit: Calendar missing aria role and summary

Reported by: mikeb Owned by: bill
Priority: undecided Milestone: 1.8
Component: Dijit Version: 1.7.2rc1
Keywords: Cc:
Blocked By: Blocking:

Description

dijit.Calendar and CalendarLite? use role="columnheader". However, the required context for a columnheader is a parent with the role="row". (http://www.w3.org/TR/wai-aria/roles#columnheader). The <tr elements in the thead need role="row".

Also, no meaningful summary for the table is provided that describe the purpose of the table. This is getting picked up by our automated tools because the header cells of the calendar (week days) are not in the first row of the table.

Generally it is the responsibility of the developer using the calendar widget to provide enough information on the page for the user to understand that they are interacting with a calendar. We don't want to add a generic summary attribute to the calendar that says, "calendar" as this would be annoying for JAWS users to hear each time they used the calendar.

In lieu of this I think Dojo should just provide a default empty summary attribute. This encourages developers to use the summary if they deem is appropriate for their users.

Attachments (1)

calendar.a11y.patch (1.6 KB) - added by mikeb 7 years ago.

Download all attachments as: .zip

Change History (3)

Changed 7 years ago by mikeb

Attachment: calendar.a11y.patch added

comment:1 Changed 7 years ago by bill

Component: CoreDijit
Milestone: tbd1.8
Owner: set to bill
Status: newassigned

Specify summary to Calendar will already apply it to the this.domNode, but I'll checkin the role fix.

comment:2 Changed 7 years ago by bill

Resolution: fixed
Status: assignedclosed

In [28437]:

Add aria role, fixes #15257, thanks Mike !strict.

Note: See TracTickets for help on using tickets.