-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Accessibility fix generate access code #2783
Accessibility fix generate access code #2783
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Thanks for all the PRs @galupa. I'll get through them all sometime this week |
@@ -50,6 +50,36 @@ $(document).on('turbolinks:load', function(){ | |||
showDeleteRoom(this) | |||
}) | |||
|
|||
// For keyboard users to be able to generate access code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as for the other PR - can this be moved into a separate function?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, thanks for reviewing the codes, I have made the changes, please check it again when you have time
tidy up code based on CR
d3c9399
to
32f6e4e
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
…#2783) tidy up code based on CR Co-authored-by: Rangga Permana <[email protected]>
…#2783) tidy up code based on CR Co-authored-by: Rangga Permana <[email protected]>
This PR should enable keyboard users to be able to generate or reset access codes. It also enable users with visual impairment to know what the generated code is.
Description
The changes on this PR are as follows:
Testing Steps
To test this:
Screenshots (if appropriate):
Not much changes on UI, but the buttons / icon should be able to be selected by only using keyboard