Opened 12 years ago

Closed 12 years ago

#4830 closed defect (fixed)

TimePicker tabIndex=1

Reported by: Douglas Hays Owned by: haysmark
Priority: high Milestone: 1.0
Component: Dijit Version: 0.9
Keywords: Cc: dante
Blocked By: Blocking:

Description

TimePicker? template has a tabIndex="1". This will cause improper tabbing.

Attachments (1)

4830.patch (1.1 KB) - added by haysmark 12 years ago.
Fixes #4830. Removed tabIndex from TimePicker?.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 12 years ago by Adam Peller

Cc: dante added

comment:2 Changed 12 years ago by dante

my fault on that one actually, in my attempts to have keyboard accessibility within the widget. i will/can fix, but i suggest we look at the possibility of having tab/arrow accessibility in popups. _Calendar as well, though current 1.0 a11y requirements have allowed us to not need it in either widget.

Changed 12 years ago by haysmark

Attachment: 4830.patch added

Fixes #4830. Removed tabIndex from TimePicker?.

comment:3 Changed 12 years ago by haysmark

Here is a patch to remove the tabIndex.

comment:4 Changed 12 years ago by dante

Resolution: fixed
Status: newclosed

(In [11099]) fixes #4830 - for 1.0 the popup part of timepicker will not be accessible. post 1.0, it will be a goal to make both _Calendar and _TimePicker keyboard accessible.

Note: See TracTickets for help on using tickets.