Skip to content

Fix accessibility of tab bar#122

Closed
dusek wants to merge 4 commits intognachman:masterfrom
dusek:accessibility-tabbar
Closed

Fix accessibility of tab bar#122
dusek wants to merge 4 commits intognachman:masterfrom
dusek:accessibility-tabbar

Conversation

@dusek
Copy link
Contributor

@dusek dusek commented Mar 31, 2013

More information in commit messages.

I will also try to submit a pull request to dorianj/PSMTabBarControl,
as I now discovered that is the source :-)

dusek added 4 commits March 31, 2013 22:35
* it's now an AXTabGroup - thanks to AXTabs the position of the tab
  and total tab count are announced
* AXChildren shows only visible tabs, and all buttons (like overflow)
* AXValue + AXValueChanged notification enable announcing new tab name
  when switching between them with Cmd-Shift-[] (for unknown reason
  does not do that quite yet)
* being AXRadioButton, VoiceOver indicates there is exclusive
  choice (i.e. only one button can be selected at a time, just like a
  radio button), also in conjunction with AXTabs of its parent reports
  proper position
* implemented visual bounds - yields proper VoiceOver cursor bounds
  and also enables touch exploration
* implemented AXTitle - so that the proper name is read when
  navigating the buttons
* implemented AXValue - so that the selected tab can announce itself
  as such (AXFocused is for elements having keyboard focus, i.e. being
  first responder, which is something quite different than being
  selected from a list of other options)
This returned one more in case an overflow menu is shown.
@dusek
Copy link
Contributor Author

dusek commented Mar 31, 2013

For the record, just submitted as dorianj/PSMTabBarControl#32

@dusek
Copy link
Contributor Author

dusek commented Apr 7, 2013

For the record again, the pull request dorianj/PSMTabBarControl#32 has been pulled

@gnachman
Copy link
Owner

Merged. Thanks!

@gnachman gnachman closed this Apr 16, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants