@SheGeeks
Posted
Great work here Lucas!
I second Rayane's suggestions and would add adjusting your character spacing on the button so that the letters aren't so spread out. You can use decimal values instead of whole numbers if that helps (i.e. .1ch
).
Some of your HTML and accessibility issues from the report can be reduced by switching the section
to a div
or by adding a visually hidden heading to section
. As noted in the accessibility area, using aria-label
attributes on your inputs would take care of the lack of labels neccesary for those who need assistance browsing the web. Take some time to look into web form accessibility to get a better understanding of why you're getting these particular errors and how other options to resolve them. There are easy solutions to implement without changing your code.
Your form validation is great! The only thing I would suggest adding is to also validate when someone tries to submit an empty form. If I do this on your solution, no error messages appear, though they should.
Last thing, you added an outline to the inputs to signify when someone is entering information. However, the challenge also requests that outline be applied to the hover states of all interactive elements. You have a hover state for your button. I'd add one for the input fields too and use the same styles that you have for active input fields.
Again, great work and really impressed with your form validation!
@Lusk1nha
Posted
@SheGeeks Thank you for your tips and to share your knowledge.