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

add aria markup to umb-toggle so it behaves like the checkbox it is #6506

Merged

Conversation

zersiax
Copy link
Contributor

@zersiax zersiax commented Oct 1, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

If there's an existing issue for this PR then this fixes

Description

@zersiax
Copy link
Contributor Author

zersiax commented Oct 1, 2019

Normally, umb-toggles are semantically rendered as buttons. They aren't buttons conceptually though, they are toggles , which is semantically represented by a checkbox. I added aria-role checkbox and a matching aria-checked property that mirrors the ng-checked state. As a bonus, umb-toggles that have a labeldefined now actually speak that label properly using screen readers.

Copy link
Contributor

@dawoe dawoe left a comment

Choose a reason for hiding this comment

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

Hi @zersiax

Congratulations on your first PR for the Umbraco CMS.

I tested and approve this one.

Dave Community Pull Request team

@zersiax
Copy link
Contributor Author

zersiax commented Oct 2, 2019

I see the azure build fails, but on first glance it doesn't seem to be because of something I did? Does anyone know more?

@zersiax zersiax closed this Oct 2, 2019
@zersiax zersiax reopened this Oct 2, 2019
@nul800sebastiaan nul800sebastiaan merged commit e0481ac into umbraco:v8/dev Oct 14, 2019
@nul800sebastiaan
Copy link
Member

Hi @zersiax and congrats on your first PR for Umbraco CMS!

The build was failing that day for everyone due to a configuration update we needed to do, this is not a failure due to your changes, no worries.

Thanks for the update on these toggles, this works well and I can indeed hear the correct label in NVDA.

Congrats again, I've added a new contributor badge to your profile on Our Umbraco as well! 👍 https://our.umbraco.com/members/zersiax

@nul800sebastiaan
Copy link
Member

Hi there @zersiax,

First of all: A big #H5YR for contributing to Umbraco during Hacktoberfest! We are very thankful for the huge amount of PRs submitted, and all the amazing work you've been doing 🥇

We have had another record breaking amount of PR's in both the CMS, Documentation, and other smaller public repositories. You can read all about the numbers in the Umbraco Hacktoberfest round-up blog post.

Due to the amazing work you and others in the community have been doing, we've had a bit of a hard time keeping up. 😅 While all of the PRs for Hacktoberfest might not have been merged yet, you still qualify for receiving some Umbraco swag, congratulations! 🎉

In the spirit of Hacktoberfest we've prepared some exclusive Umbraco swag for all our contributors - including you!

Receive your swag! 👈 Please follow this link to fill out and submit the form, before December 19th 2019.

Following this date we'll be sending out all the swag, which also means that it might not reach your doorstep before February, so please bear with us and be patient 🙏

The only thing left to say is thank you so much for participating in Hacktoberfest! We really appreciate the help!

Kind regards,
The Umbraco PR Team & The Documentation Curators

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants