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

fix typo for application #159

Closed
wants to merge 1 commit into from
Closed

fix typo for application #159

wants to merge 1 commit into from

Conversation

yuheiy
Copy link

@yuheiy yuheiy commented Aug 2, 2019

  • Remove the changed marker in Description cell
  • Remove aria-expanded from Supported Properties cell

Preview | Diff

@scottaohara
Copy link
Member

Thanks for filing the issue, @yuheiy

There does seem to be a bit of a misalignment here.

aria 1.1 notes that aria-expanded is not supported on role=application. However, aria 1.2 has it listed under the supported states and properties.

There is an issue that was just opened to clarify the allowed roles for aria-expanded, where this specific allowance was brought up. So for now it makes sense to wait on the outcome of that issue.

@siusin
Copy link
Contributor

siusin commented Aug 2, 2019

Marked as non substantive for IPR from ash-nazg.

@scottaohara scottaohara added the Do not merge A PR which needs additional work or question resolved before it can be merged. label Aug 2, 2019
@scottaohara
Copy link
Member

So, ARIA will be allowing aria-expanded on application, at least in 1.2 but I don't think this will be changed going forward either.

The changed marker has also been removed (along with all changed / new markers) via another PR. So this PR is no longer necessary.

Thank you for raising it / helping to prompt some further discussion over in the ARIA WG per this attribute allowance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not merge A PR which needs additional work or question resolved before it can be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants