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

Core-AAM Issue 492: Clarify AX Tree Exclusion for Children Presentational True #647

Merged
merged 2 commits into from
Sep 15, 2017

Conversation

mcking65
Copy link
Contributor

@mcking65 mcking65 commented Sep 15, 2017

For issue #492, modified section "5.1.1 Excluding Elements from the Accessibility Tree" in core-aam.html.

Clarified the criteria for elements with children presentational true to specify that:

  1. Text content of excluded elements is included and;
  2. Exceptions, such as focusability, also apply.

…onal Elements

For issue #492, modified section "5.1.1 Excluding Elements from the Accessibility Tree" in core-aam.html.

Clarified the criteria for elements with children presentational true to specify that:
1. Text content of excluded elements is included and;
2. Exceptions, such as focusability, also apply.
@mcking65 mcking65 changed the title Core-AAM Issue 492: Clarify AX Tree Exclusion for Children Presentati… Core-AAM Issue 492: Clarify AX Tree Exclusion for Children Presentational True Sep 15, 2017
Copy link
Contributor

@joanmarie joanmarie left a comment

Choose a reason for hiding this comment

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

Thanks for doing this! I wonder if the new content would read better if the new text came after the list of elements. Or alternatively putting something (a paragraph break, some transitional wording) in between "is included." and "Elements having." What do you think?

@mcking65
Copy link
Contributor Author

Not liking the details after the list, but breaking into multiple paragraphs seems like it might help. Does commit 7612372
work for you?

@joanmarie joanmarie merged commit 19d5fef into master Sep 15, 2017
@joanmarie
Copy link
Contributor

I dig it. Merged. Thanks much!!

@joanmarie joanmarie deleted the issue492 branch April 18, 2018 17:13
pkra pushed a commit that referenced this pull request May 20, 2024
… in Core-AAM (#647)

Clarify the criteria for elements with children presentational true to specify that:
1. Text content of excluded elements is included
2. Exceptions, such as focusability, also apply

This is already stated in the ARIA specification, but was not clear from
the text present in the Core AAM.

Fixes github issue #492.
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.

2 participants