Opened 12 years ago

Closed 12 years ago

#3071 closed defect (fixed)

spinner a11y is broken

Reported by: Douglas Hays Owned by: Sam Foster
Priority: high Milestone: 0.9beta
Component: Accessibility Version: 0.9
Keywords: Cc: Becky Gibson
Blocked By: Blocking:

Description

With revision [8660], spinner now has huge problems. Using FF 2.0 on WinXP:

ctrl + does not resize the arrows or their containing boxes.

A large font size used by visually imparied users does not resize the arrows accordingly.

High contrast mode does not show arrows at all.

With images off, the arrows do not show up at all.

These scenarios were all working well before this revision.

Attachments (3)

HC_ie7.JPG (3.0 KB) - added by Becky Gibson 12 years ago.
combobutton.jpg (5.4 KB) - added by Becky Gibson 12 years ago.
largespinner.jpg (7.2 KB) - added by Becky Gibson 12 years ago.

Download all attachments as: .zip

Change History (19)

comment:1 Changed 12 years ago by bill

Owner: changed from Becky Gibson to owen

comment:2 Changed 12 years ago by Douglas Hays

Cc: Becky Gibson added; owen removed

comment:3 Changed 12 years ago by owen

(In [8699]) refs #3071 #2839 One possible workaround for the arrow-image problem (3071), as well as Tundraification of the Progress bar. See those bugs for more details

comment:4 Changed 12 years ago by owen

Checked in (to Autocompleter only) is another possible solution in terms of having a non-image-based-arrows that will scale. I'll send mail out tonight about the possible approaches and what I've learned. This will likely change again soon. (Note: no actual effect on Spinner at this time, this bug is for all of these types of problems).

comment:5 Changed 12 years ago by owen

(In [8701]) refs #3071 Arrows are at least on the same line in IE, although they're the same size. Problem was that width and height need to be explicitly set on the fieldset in IE to make the nested fieldsets work.

comment:6 Changed 12 years ago by owen

Status: newassigned

comment:7 Changed 12 years ago by owen

Owner: changed from owen to Becky Gibson
Status: assignednew

Becky, can you verify what still needs to be done and reassign to me? Thanks

comment:8 Changed 12 years ago by Becky Gibson

In consistent colors in high contrast mode on IE 7 Windows. Arrow and button text matches high contrast colors for combo button but not for other buttons (they are white text and arrow). It is ok on Firefox. See HC_ie.jpg attached.

Arrows shift when change text size in FF. I can live with this since IE 7 allows scaling and Firefox 3 will provide scaling as well. See largespinner.jpg

Combobuttons have sizing issues when font is scaled in FF. I had to increase size 3 times to see this - can probably live with if necessary See combobutton.jpg

accordionContainer arrows need to be updated to use new method - currently they do not work in high contrast and images off mode (can make a separate ticket).

Need to identify the active tab - perhaps with a border - right now it is just a background color.

Changed 12 years ago by Becky Gibson

Attachment: HC_ie7.JPG added

Changed 12 years ago by Becky Gibson

Attachment: combobutton.jpg added

Changed 12 years ago by Becky Gibson

Attachment: largespinner.jpg added

comment:9 Changed 12 years ago by Becky Gibson

Note that combobuttons are ok after resizing if you reload the page - this is acceptable.

Screen reader now speaks "question" for the down arrow on the dropdown button - separate ticket filed as there is a work around to fix this. Screen reader is speaking "question" for the html entity character. Generally these characters will not get focus and thus will not be spoken. The button is hopefully the only place where this will be an issue. #3032

comment:10 Changed 12 years ago by Becky Gibson

Owner: changed from Becky Gibson to owen

comment:11 Changed 12 years ago by owen

Status: newassigned

comment:12 Changed 12 years ago by bill

Owner: changed from owen to samian
Status: assignednew

Assigning to Sam Foster since he's working on it.

About focus on the spinner buttons: I don't think they should be able to receive focus. Keyboard support for spinner widget is done by using the up/down arrow keys (or by typing a number directly).

comment:13 Changed 12 years ago by bill

Owner: changed from samian to Sam Foster

comment:14 Changed 12 years ago by Sam Foster

Status: newassigned

comment:15 Changed 12 years ago by Sam Foster

See patch attached to #3301 to address this ticket also. I've been able to test high contrast mode, but not screen-readers, however I've followed the implementation of other form widgets and am reasonably confident its good there.

comment:16 Changed 12 years ago by Sam Foster

Resolution: fixed
Status: assignedclosed

Got a verbal ok from Becky that this checks out as fixed in Jaws, high-contrast mode. Focus handling is correct and resizing/font-scaling works. The patch (attached to #3301) has been merged, and it looks complete here, so I'm going to close this. Please re-open if I've misunderstood or there's still an issue.

Note: See TracTickets for help on using tickets.