@VCarames
Posted
Hey @MelvinAguilar. Great Job on this landing page!
-
The Picture Element is not really necessary for this challenge since you are not changing the image for a different sized one or changing the contrast/brightness when applying a dark theme.
-
This one depends on the company's/developer's preference but you can leave the Alt Tags empty for the illustrations and a aria-hidden=โtrueโ since they are just decorative.
-
Instead of wrapping each "card feature" in an Article Element it would better to use a Div instead, since none of the cards can stand on their own. The way you can determine if something can be wrapped in an Article Element is if it can be placed on a random website or a blank one and still make sense on its own.
Keep up the good work! Happy Coding! ๐ป๐
Marked as helpful
@MelvinAguilar
Posted
@vcarames
Hey, there! ๐ Thank you very much for your comments and the time you spent analyzing my code! ๐ I will fix that as soon as possible.