Design comparison
SolutionDesign
Solution retrospective
Hey everyone I would love for someone to take a look at my code, let me know if think there are redundant or unnecessary code. thanks.K
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
LABELS 🔖:
input
elements are having alabel
with empty content, those labels are not gonna work.
- Instead you can use the html native attribute named
aria-label
- A
aria-label
is used to create a caption for a form control. Thearia-label
can be associated with a form control either implicitly by placing the control element inside the label element, or explicitly by using the for attribute
- Effective form
labels
are required to make forms accessible. The purpose ofform
elements such ascheckboxes
,radio
buttons,input
fields, etc, is often apparent to sighted users
- Even if the
form
element is not programmatically labeled. Screen readers users require useful formlabels
to identifyform
fields, So here usingaria-label="{values}"
attributes forinput
is enough to be accessible
- Example:
<input type="text" aria-label="First Name" placeholder="First Name" id="first-name" name="first-name" class="registration-form__input">
- Now you can remove those
label
elements withsr-only
classes, usingaria-label
makes the code clean also
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin 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