Opened 13 years ago

Last modified 13 years ago

#2664 closed defect

a11y improvements for dijit inlineEditbox — at Version 4

Reported by: Becky Gibson Owned by: ptbrunet
Priority: high Milestone: 0.9beta
Component: Accessibility Version: 0.9
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Becky Gibson)

The inlineEditbox needs an ARIA role - specifically the dojoAttachPoint=editable element (which has tabindex=0) needs a role. Assuming that this will always contain a input type=text element, the proper role is textfield. This can be added to the template via waiRole=textfield. This role is needed so the that screenreader will recognized this as an editable field even when it is not being edited.

When the inlineEditBox is disabled, it needs the property of readonly="true" specified. When it is enabled, the property must be removed or set to readonly="false".

Instructions for using this widget should also recommend attaching a label to the editable text. This may be a bit difficult for an inline edit field but is important for screen reader users and should be included whenever possible.

This part of the ticket is no longer necessary. Due to the way the inline editing has been implemented as a wrapper around any type of control we can't easily enable this level of keyboard support. [While there are buttons provided to save and cancel an inline edit, I would also like to see the keyboard implemented for these actions. When editing an inline field pressing escape should cancel the edit and enter should accept the edit. Since the buttons are provided we can technically live with out this but it would make the control more usable for keyboard only and screen reader/screen magnifier users.]

Change History (5)

comment:1 Changed 13 years ago by Becky Gibson

Description: modified (diff)
Status: newassigned

comment:2 Changed 13 years ago by Becky Gibson

Description: modified (diff)

comment:3 Changed 13 years ago by Becky Gibson

Owner: changed from Becky Gibson to ptbrunet
Status: assignednew

comment:4 Changed 13 years ago by Becky Gibson

Description: modified (diff)

Changed 13 years ago by ptbrunet

Attachment: InlineEditBox.patch added

patch adding wai role button to dijitform emplatesInlineEditBox.html

Note: See TracTickets for help on using tickets.