Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Document new tabIndex property for radio component #2076

Closed
blackbaud-johnly opened this issue Oct 4, 2018 · 1 comment
Closed

Document new tabIndex property for radio component #2076

blackbaud-johnly opened this issue Oct 4, 2018 · 1 comment

Comments

@blackbaud-johnly
Copy link
Contributor

blackbaud-johnly commented Oct 4, 2018

blackbaud/skyux-forms#2

The tabIndex input for the radio component has been deprecated in favor of using the new tabIndex input on the radio group component. The tabIndex represents the position an element is within tab navigation on the page. (tab navigation is when you press tab or shift+tab and focus shifts between elements on the page). A tabIndex of -1 removed the element from tab navigation entirely.

According to accessibility guidelines, focus on a group of radio elements should only ever focus the currently selected radio button or if one isn't selected, the first/last button depending on the direction navigated from. So the tabIndex of a group of radio buttons has been moved to the radio-group element, since individual tabIndexes do not make sense.

@blackbaud-johnly
Copy link
Contributor Author

Addressed in blackbaud/skyux2-docs#200.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants