@Drougnov
Posted
Hello @dombrga, Great job. I like how you added different error messages for empty and invalid emails. And good job on the responsive part too.
However here are something you might wanna consider:
-
The Facebook icon doesn't match the size of the other icons. Add some padding to it.
-
To fix accessibility issues, add
aria-label='something like this'
to thosea
tags. -
To fix Html issues, use div instead of section. Cause you need headings while using section or article tags.
Overall, it's looking great. And it's my favorite solution to this challenge. Have a good day :)