-
Notifications
You must be signed in to change notification settings - Fork 231
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
A11y fixes #36
Open
lloydi
wants to merge
18
commits into
pkalogiros:production
Choose a base branch
from
lloydi:a11y-fixes
base: production
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
A11y fixes #36
Commits on Jul 19, 2020
-
Removed tabindex="-1" from all buttons
This was making the whole UI inaccessible for keyboard-only users
Configuration menu - View commit details
-
Copy full SHA for 3b1b923 - Browse repository at this point
Copy the full SHA 3b1b923View commit details -
Provided button labels with aria-labelledby
All buttons were missing an accessible name (the inner content was being stripped by the tooltip effect. Fixed by adding id to the inner <span> and referencing it with aria-labelledby
Configuration menu - View commit details
-
Copy full SHA for 5a7c436 - Browse repository at this point
Copy the full SHA 5a7c436View commit details -
Allows for screen reader users to jump from region to region more easily
Configuration menu - View commit details
-
Copy full SHA for 812116d - Browse repository at this point
Copy the full SHA 812116dView commit details -
Configuration menu - View commit details
-
Copy full SHA for c605c64 - Browse repository at this point
Copy the full SHA c605c64View commit details -
Configuration menu - View commit details
-
Copy full SHA for dec6633 - Browse repository at this point
Copy the full SHA dec6633View commit details -
Made 'load sample' link accessible
Changed wording (should avoid 'click here phrases', and link phrase now makes sense out of context ("load a sample audio file")
Configuration menu - View commit details
-
Copy full SHA for b8d4977 - Browse repository at this point
Copy the full SHA b8d4977View commit details -
Configuration menu - View commit details
-
Copy full SHA for 5a2d30c - Browse repository at this point
Copy the full SHA 5a2d30cView commit details -
Added focus styles for all main controls
Doubled up on all hover styles with :focus equivalent Added new styles for all .pk_btn buttons
Configuration menu - View commit details
-
Copy full SHA for dcc4d1b - Browse repository at this point
Copy the full SHA dcc4d1bView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ca8cb0 - Browse repository at this point
Copy the full SHA 8ca8cb0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 50886c3 - Browse repository at this point
Copy the full SHA 50886c3View commit details
Commits on Jul 20, 2020
-
Configuration menu - View commit details
-
Copy full SHA for ef00921 - Browse repository at this point
Copy the full SHA ef00921View commit details -
Configuration menu - View commit details
-
Copy full SHA for c42cb40 - Browse repository at this point
Copy the full SHA c42cb40View commit details -
Set focus on OK button on load Changed link phrase for Githib so link makes sanes out of context
Configuration menu - View commit details
-
Copy full SHA for 2587ebe - Browse repository at this point
Copy the full SHA 2587ebeView commit details -
Configuration menu - View commit details
-
Copy full SHA for 4c900cf - Browse repository at this point
Copy the full SHA 4c900cfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 6f8eea3 - Browse repository at this point
Copy the full SHA 6f8eea3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2cf2e55 - Browse repository at this point
Copy the full SHA 2cf2e55View commit details -
Configuration menu - View commit details
-
Copy full SHA for ac409e8 - Browse repository at this point
Copy the full SHA ac409e8View commit details -
Configuration menu - View commit details
-
Copy full SHA for 2921d76 - Browse repository at this point
Copy the full SHA 2921d76View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.