Opened 11 years ago

Closed 4 years ago

#7866 closed defect (patchwelcome)

Spinner: inconsistency with null value and aria-valuenow property

Reported by: Joseph Scheuhammer Owned by:
Priority: high Milestone: 1.13
Component: Accessibility Version: 1.2.0
Keywords: spinner aria valuenow Cc: davidb
Blocked By: Blocking:

Description

When a spinner is set to a null value via spinner.attr('value', null), the value of the spinner and its aria-valuenow property do not match:

spinner.attr('value') -> NaN

dijit.getWaiState(spinner, 'valuenow') -> "" (the empty string).

The ARIA specification for spinner is not clear on this case: see http://www.w3.org/WAI/PF/aria/#spinbutton.

FWIW, I think the spinner's value and its aria-valuenow should always match. In this case, aria-valuenow should also be NaN.

Ultimately, it depends on what the ARIA working group decides.

Change History (8)

comment:1 Changed 11 years ago by Joseph Scheuhammer

Component: a11y - reviewAccessibility
Owner: changed from becka11y to Becky Gibson

comment:2 Changed 11 years ago by Joseph Scheuhammer

Noting that the ARIA specs on "valuenow" imply that if the value is unknown, then valuenow should be absent. Quoting from http://www.w3.org/WAI/PF/aria/#valuenow:

If the value is not known (as often occurs with progress bars) then the valuenow attribute should not be set at all. If the valuenow attribute is absent, no information is implied about the current value.

comment:3 Changed 11 years ago by Becky Gibson

Note that when this is fixed, the tests/form/robot/test_Spinner.html test should be updated to include a test for a null aria-valuenow. The current test is commented out.

comment:4 Changed 11 years ago by Becky Gibson

Milestone: tbdfuture

will address this for next Firefox release - moving to future

comment:5 Changed 9 years ago by Chris Mitchell

Owner: changed from Becky Gibson to Douglas Hays

reassign due to inactive committer. please triage accordingly.

comment:6 Changed 6 years ago by Douglas Hays

Owner: Douglas Hays deleted
Status: newassigned

comment:7 Changed 6 years ago by Douglas Hays

Status: assignedopen

comment:8 Changed 4 years ago by dylan

Milestone: future1.12
Resolution: patchwelcome
Status: openclosed

A pull request would be welcome for this, but given the lack of activity over the past 8 years, I'm closing this one.

Note: See TracTickets for help on using tickets.