El Ghali Khalid• 2,630
@limtedtorni000
Posted
Very good Its look great and very responsive keep contributing these awesome solutions
1
Jane• 1,040
@janegca
Posted
@limtedtorni000, thank you for the kind words.
0
Design comparison
Solution retrospective
Any suggestions for improvements welcome.
Community feedback
Paritosh Sahni• 80
@Paritosh7
Posted
Hey Janegca! That's a cool README.md you have for this project!
0
Really nice solution, well done!
0
One question I have about the form actually...
I always thought error messages needed to be inside an ID-ed element that contains the aria-live attribute with a value of polite, and is linked to the input using aria-desciribedby (on the input pointing to that id). I'd have to test how this works with my screenreader, as I've never seen error messages placed inside a label element before. Have you tried that out?
0
Jane• 1,040
@janegca
Posted
Hi @grace-snow, thank you for taking the time to comment. I haven't tried my solution with a screen reader. I know very little about accessability, honestly, just used the aria labels because I got dinged on an earlier challenge for not using them. I really need to read more about it.
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord