Exclude "Show info" button from a11y report
See original GitHub issueIs your feature request related to a problem? Please describe. Will be nice, if “Show info” button (@storybook/addon-info) will not have effect to a11y report (@storybook/addon-a11y). Currently it always spawn this violation:
.info__show-button Element has insufficient color contrast of 3.84 (foreground color: #ffffff, background color: #2288cc, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
Describe the solution you’d like
May be aria-hidden
to “Show info” button.
Describe alternatives you’ve considered A clear and concise description of any alternative solutions or features you’ve considered.
Are you able to assist bring the feature to reality? yes, maybe
Issue Analytics
- State:
- Created 4 years ago
- Reactions:1
- Comments:6 (5 by maintainers)
Top Results From Across the Web
Exclude "Show info" button from a11y report #6256 - GitHub
If there's a quick fix please feel free to submit a PR and I'd be happy to merge it. However we're moving the...
Read more >Accessibility and usability considerations for disabling buttons ...
HTML buttons and inputs accept a Boolean disabled attribute. ... Can we remove it from the form and only show it when it's...
Read more >Checklist - The A11Y Project
This includes things like inactive drop down menus, off screen navigations, or modals. Share Link to checklist item: Remove invisible focusable elements.
Read more >aria-hidden - Accessibility - MDN Web Docs - Mozilla
The aria-hidden state indicates whether the element is exposed to an accessibility API.
Read more >Easy Checks – A First Review of Web Accessibility - W3C
To get the hidden information, click the button or click anywhere on the heading. The sections below all have hidden information under expandable...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
ZOMG!! I just released https://github.com/storybooks/storybook/releases/tag/v5.0.6 containing PR #6301 that references this issue. Upgrade today to try it out!
If there’s a quick fix please feel free to submit a PR and I’d be happy to merge it. However we’re moving the info addon out of the component preview in the very near future, so you might want to just wait for that.