@VCarames
Posted
Hey @KevinMA95, adding to what was said above/below:
- To give you HTML code structure, you want to set up your code in the following manner (only did parent containers):
HTML
<body>
<main>
<article class=“card-container”>
<picture></picture>
<div class="card-content">
</div>
</article>
</main>
</body>
The Main Element identifies the main content of the document.
While the Article Element will serve as the card’s container, because the card represents a complete, or self-contained, section of content that is, in principle, independently reusable.
More info:
https://web.dev/learn/html/headings-and-sections/
-
The Alt Tag Description for the image needs to be improved upon. You want to describe what the image is; they need to be readable. Assume you’re describing the image to someone.
-
The old price is not being announced properly to screen readers. You want to wrap it in a Del Element and include a sr-only text explaining that this is the old price.
-
Your "button" was created with the incorrect element. When the user clicks on the button they should directed to a different part of you site. The Anchor Tag will achieve this.
Happy Coding! 👻🎃
Marked as helpful
@KevinMA95
Posted
Hello @vcarames, Thank you so much for the tips and explanations they were really helpful also I didn't notice I was missing the <a> tag in my button, thanks for pointing that out.
Happy coding!