Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revisions to accessible name required #1477

Merged
merged 2 commits into from Jul 2, 2021
Merged

Revisions to accessible name required #1477

merged 2 commits into from Jul 2, 2021

Conversation

scottaohara
Copy link
Member

@scottaohara scottaohara commented Apr 29, 2021

closes #1466

  • Removes instances of "accessible name required: false" - only when accName is required will this row have a "True" value.
  • Requires tab have an accessible name.
  • Removes requirement that a tooltip needs an accessible name.

Note: #1465 is related to this issue and accepting that PR will mark form as being marked as requiring an accessible name in sections such as 5.2.8.4 Roles Supporting Name from Author


Preview | Diff

removing instances of “accname required false” to be consistent with other roles which do not require a name, but do not state “false”
- require a name for `tab`
- remove requirement for a name from `tooltip`
Copy link
Contributor

@carmacleod carmacleod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1
Totally agree with these changes.

@jnurthen jnurthen merged commit b41a010 into main Jul 2, 2021
jnurthen pushed a commit that referenced this pull request Jul 2, 2021
* remove instances of “false”
* tab / tooltip req accname updates
  - require a name for `tab`
  - remove requirement for a name from `tooltip`
@scottaohara scottaohara deleted the so_1466 branch July 13, 2021 11:25
@pkra pkra added this to the ARIA 1.3 milestone Jan 12, 2022
@pkra pkra mentioned this pull request Jan 12, 2022
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.

Roles with required accessible names
6 participants