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

Removing "golden hammer" language #354

Closed
wants to merge 1 commit into from
Closed

Conversation

bpmcneilly
Copy link
Contributor

The 'golden hammer', removing elements with role="none" or "presentation" entirely from the accessibility tree, was rejected by the task force so I removed this text.

For stylistic reasons I removed "in addition" and added the sentence beginning "Certain roles" to the previous paragraph.

The 'golden hammer', removing elements with role="none" or "presentation" entirely from the accessibility tree, was rejected by the task force so I removed this text.

For stylistic reasons I removed "in addition" and added the sentence beginning "Certain roles" to the previous paragraph.
@AmeliaBR
Copy link
Contributor

Thanks for noticing this section, Brian. It is problematic given the current interpretation.

That said, I'm not sure whether we really need to remove all that text, or whether we should keep it with new qualifications. Describing how role=presentation normally works might still be useful, although perhaps it is less relevant since this is also the default behavior. I'll take another look at it later, maybe integrate the text elsewhere.

@bpmcneilly
Copy link
Contributor Author

Good point. That sentence about presenting every element is still true (regardless of the golden hammer), but I couldn't gracefully find a way to integrate it in this section.

Perhaps this should find its way into the authoring practices document? Just a thought.

@AmeliaBR
Copy link
Contributor

AmeliaBR commented May 7, 2016

Closing in favour of more comprehensive changes proposed in #365.

@AmeliaBR AmeliaBR closed this May 7, 2016
pkra pushed a commit that referenced this pull request May 20, 2024
* revise mapping tables for svg and math

closes #344

* removing some unintended whitespace changes

* removing more unintended whitespace changes

* Update index.html
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