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

Fallback table needs a language update to reflect presence of non-required attributes #949

Closed
melanierichards opened this issue Apr 11, 2019 · 4 comments · Fixed by #1067
Labels
has PR PR exists that will close this issue
Milestone

Comments

@melanierichards
Copy link
Contributor

Some updates in the 1.2 milestone involve moving ARIA properties from "required" to "supported" on various roles. An example is aria-valuemin, aria-valuemax on the slider role (#860). With these changes, the language for the table entitled "Fallback values for missing required attributes" in Section 9.2 no longer agrees with its contents, as there are both required and supported properties in the table.

We should either change the language of Section 9.2, or drop the UA requirements for the supported attributes (IIRC we did not have plans to change UA expectations for these). The table title and references would need updating, as well as the "Required Attribute" column header.

@melanierichards
Copy link
Contributor Author

Required Properties with Default Values wiki page for a reference of the changes in this project

@joanmarie joanmarie added this to the ARIA 1.2 milestone Apr 11, 2019
@scottaohara
Copy link
Member

I'll take a stab at this.

@melanierichards
Copy link
Contributor Author

On the 2019-05-23 call, the group decided to remove the supported but not required attributes from this table. Some of this cleanup will occur as a consequence of new PRs in this project, like #946, but we should keep this issue open to do a cleanup pass.

@melanierichards
Copy link
Contributor Author

Oh I do want to add that it's a little odd to have to look in multiple places to find the repair techniques for supported attributes. So we'll want to at least add some sentence around 9.2 telling UAs to check the roles for implicit values on supported attrs (MAY statement at most).

@scottaohara scottaohara removed their assignment Sep 27, 2019
@jnurthen jnurthen added the has PR PR exists that will close this issue label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has PR PR exists that will close this issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants