@VCarames
Posted
Hey there!👋 Here are some suggestions to help improve your code:
Regarding you questions:
- There is no shadow.
- The report is designed for a "full sites". It can't tell that this is just a component. The way you did it, is correct.
-
The purpose of the Main Element is to identify the main content of your page. It is not the container of you component. After the main element, you want add a container to wrap you separate components in.
-
The car images/icons in this component are purely decorative; They add no value. So their Alt Tag should left blank and have an aria-hidden=“true” to hides them from assistive technology.
If you have any questions or need further clarification, let me know.
Happy Coding! 👻🎃
Marked as helpful
@Carmendelber
Posted
@vcarames Thanks for your comment!! btw, could sections be used as containers then?
@VCarames
Posted
@Carmendelber
The section element would work, but I would suggest using the article element instead. Since this component is like a "flyer", it makes sense on its own and can be used on other sites, so the article elements make more sense.
Marked as helpful
