-
Notifications
You must be signed in to change notification settings - Fork 136
Add and link attribute terms: Defines, Identifies, and Indicates. #1439
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
Conversation
Confession: I found it somewhat challenging to define the terms Defines, Identifies, and Indicates, without using those words in the definitions. ;-) I went with "denote" but I'm very open to nitpick changes if a better term is suggested. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - note that terms is maintained in aria-common so these changes will need removing from this PR and creating there - but I can do that when I merge it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! "Denote" works for me. :)
I had one nit for "type is of type" in Defines.
Regarding the XML Schema terms vs. the ARIA Characteristics Table type terms, I'll leave that up to you.
@carmacleod I think all your requested changes are incorporated now. Please verify and thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Good job on the boolean-token wording - you got the point across perfectly.
Thanks for persevering. :)
Accepting CM’s redundancy correction. Co-authored-by: Carolyn MacLeod <[email protected]>
) * Add and link attribute terms: Defines, Identifies, and Indicates. Resolves #516. Co-authored-by: Carolyn MacLeod <[email protected]> Co-authored-by: James Nurthen <[email protected]>
Add and link attribute terms: Defines, Identifies, and Indicates. Resolves #516.
Preview | Diff