Skip to content

Accessibility fixes (and tests) #1227

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.

Sign up for GitHub

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

Merged
merged 24 commits into from
Jan 17, 2018
Merged

Accessibility fixes (and tests) #1227

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
0929810
fix(templates/application): Add labels to search input fields
sorin-davidoi Jan 9, 2018
c00715f
fix(templates/application): Logo should not be focusable
sorin-davidoi Jan 9, 2018
dd877cc
fix(styles/home/button): Use hover style for focus
sorin-davidoi Jan 9, 2018
d3ff8cd
fix(templates/application): Use semantics HTML tags (nav, main, footer)
sorin-davidoi Jan 9, 2018
bd4c191
fix(templates/index): Use section for each crate list
sorin-davidoi Jan 9, 2018
89c8ede
fix(templates/components/crate-list): Use ol instead of ul
sorin-davidoi Jan 9, 2018
3e94117
fix(templates/application): Remove tabindex="1" from search fields
sorin-davidoi Jan 9, 2018
f0c8377
fix(templates/application): Add alt text for the "For me on GitHub" link
sorin-davidoi Jan 9, 2018
72407de
fix(styles/app/button): Do not set outline: 0
sorin-davidoi Jan 9, 2018
522eb3e
fix(styles/me): Set focus style for load more button
sorin-davidoi Jan 9, 2018
1aa4b29
fix(templates/application): Move fork button to the end of the document
sorin-davidoi Jan 9, 2018
ea4fb9c
fix(componentns/user-avatar): Add alt text (name, username)
sorin-davidoi Jan 9, 2018
961ba48
fix(crate/version): Nest li directly inside ul
sorin-davidoi Jan 9, 2018
7c58178
fix(templates/crate/version): Add title to copy-to-clipboard button
sorin-davidoi Jan 9, 2018
c27533d
feat(templates/crate/version): Use section for readme and authorship
sorin-davidoi Jan 9, 2018
4a36d50
fix(templates/crate/owners): Add label to username input field
sorin-davidoi Jan 9, 2018
274c841
chore(npm/dev-dependencies): Install ember-a11y-testing
sorin-davidoi Jan 10, 2018
ed4333a
test(acceptance): Create accessibility tests
sorin-davidoi Jan 10, 2018
1ff0a23
feat(config/environment): Disable accessibility audit in during devel…
sorin-davidoi Jan 10, 2018
fc5a680
fix(templates/index): Use aria-busy and show loader using CSS
sorin-davidoi Jan 10, 2018
2cd5ac1
fix(templates/crates): Make pagination accessible
sorin-davidoi Jan 10, 2018
8d2157a
fix(templates/crate/version): Change aria-label to Crate metadata
sorin-davidoi Jan 14, 2018
e7a8209
fix(styles/home): Update custom home crate style from div to section
sorin-davidoi Jan 14, 2018
0682d78
fix(themes/application): Invalid markup
sorin-davidoi Jan 17, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion app/templates/crate/version.hbs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@
<code id="crate-toml">{{ crate.name }} = "{{ currentVersion.num }}"</code>

{{#if (is-clipboard-supported)}}
{{#copy-button clipboardTarget="#crate-toml" success=(action 'copySuccess') error=(action 'copyError')}}
{{#copy-button clipboardTarget="#crate-toml" success=(action 'copySuccess') error=(action 'copyError') title="Copy to clipboard"}}
{{svg-jar "copy" alt="Copy to clipboard"}}
Copy link
Member

@carols10cents carols10cents Jan 14, 2018 •
edited
Loading

Choose a reason for hiding this comment

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

The title on the button isn't repetitive with the alt text on the svg inside the button? I really don't know how screenreaders will read this, just wanted to point it out :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

aXe flagged this as an error, and Orca (the screen reader I test with) didn't read the text from the svg.

{{/copy-button}}
{{/if}}
Expand Down