@mattstuddert
Posted
Nice work on this challenge and congrats on submitting your first solution! 👍
One suggestion I'd have is to use a h1
element for the "We are launching soon!" title, as this is the main heading on the page. Using span
elements to wrap content doesn't create a hierarchy, which is bad for SEO and also accessibility.
I hope this helps. Keep up the great work!
@Aquelehomem
Posted
@mattstuddert Thanks matt! I just corrected the little mistake, for some reason i felt compelled to complicate things instead of just using a h1, gotta get good on the SEO thing.
@mattstuddert
Posted
@Aquelehomem you're welcome! Yeah, learning about SEO is a good bonus. Also, learning about accessibility is a must for a front-end developer. I hope you enjoyed the challenge!