Opened 12 years ago

Closed 12 years ago

#1561 closed Enhancement (wontfix)

Better a11y support

Reported by: LucaFerretti Owned by: charles
Priority: Low Milestone: None Set
Component: GTK+ Client Version: 1.40
Severity: Normal Keywords: HELPWANTED accessibility
Cc: hadess@…

Description

There are several a11y issues with current code (and maybe I've missed other).

1) custom cellrender is ignored by screen reader.

This is the issue that will require more attention, 'cause it's related to a custom widget.

If you run Orca screen reader (with braille or speech) and you move the focus on torrent list entry, orca says nothing. I think this is bad for visually impared people, but honestly I don't know what exaclty it should speak (torrent name AND statistics? dunno)

2) invalid widgets in main window

Running the AT-SPI validator prodided by Accerciser, the following errors are listes:

  • text entry in filterbar -> missing name or label
  • scrollbar (the one inside scroll pane) -> is not focusable or selectable

3) invalid widgets in preferences dialog

  • Current tab -> page tab has focused state without focusable state
  • Destination combobox -> actionable combo box is not focusable or selectable (this is strange, honestly I can focus it using Tab or shortcut, maybe the issue in in the GtkWidget?...)
  • Destination menu -> interactive menu is not actionable (the menu opened by previous combo box)
  • Destination menu -> menu missing name or label
  • Automatically Add combobox -> same issues of previus combo, plus of course the issues with related menu
  • Start_time bandwidth limit -> as previous (plus related menu)
  • Stop_time bandwidth limit -> as previous (plus related menu)
  • limits spin buttons (both) -> missing name or label
  • Proxy type combobox -> as previous (plus related menu)

4) invalid widgets in properties dialog

  • page tab has focused state without focusable state
  • Comment textentry in Comment tab -> text missing name or label
  • scrollbar in Files tab -> actionable scroll bar is not focusable or selectable
  • spin buttons in Options tab -> missing name or label

5) other

Maybe, but better read A11y devel guide before :)

Change History (8)

comment:1 Changed 12 years ago by charles

  • Priority changed from Normal to Low

Luca, can you come up with a patch for these?

comment:2 follow-up: Changed 12 years ago by charles

Luca: the reason I ask you for a patch is because:

  • Your patches in the past have all been pretty good (and accepted without changes, IIRC)
  • You've got better knowledge of a11y issues than me
  • I'm already really busy and could use the help. :)

Also, about the "5) Other" list item: please don't create open-ended tickets. I'm not going on a hunting expedition through the a11y spec just because you asked me to. Specific details are more helpful.

comment:3 Changed 12 years ago by charles

  • Type changed from Bug to Enhancement

comment:4 in reply to: ↑ 2 ; follow-up: Changed 12 years ago by LucaFerretti

Replying to charles:

Luca: the reason I ask you for a patch is because:

  • You've got better knowledge of a11y issues than me

Are you really sure???? I'll try to do something, however.

Also, about the "5) Other" list item: please don't create open-ended tickets. I'm not going on a hunting expedition through the a11y spec just because you asked me to. Specific details are more helpful.

Whops, sorry forgot to add the text. Simply I didn't run Accerciser against Create New and Log Messages dialog. So there could be other invalid widgets.

comment:5 in reply to: ↑ 4 Changed 12 years ago by charles

Replying to LucaFerretti:

Replying to charles:

Luca: the reason I ask you for a patch is because:

  • You've got better knowledge of a11y issues than me

Are you really sure???? I'll try to do something, however.

I'm sure that I have nearly zero free time anymore and actually need to spend less time on Transmission, not more, so Transmission is going to need more helpful volunteers like you. :)

comment:6 Changed 12 years ago by charles

  • Keywords HELPWANTED accessibility added

comment:7 Changed 12 years ago by hadess

  • Cc hadess@… added

comment:8 Changed 12 years ago by charles

  • Resolution set to wontfix
  • Status changed from new to closed

This is a snakepit of a ticket. If someone has specific diffs that I can apply, please attach them and reopen the ticket. But I don't see much use in vague, open-ended tickets like this one currently is.

Note: See TracTickets for help on using tickets.