Melvin Aguilar ๐ง๐ปโ๐ปโข 61,220
@MelvinAguilar
Posted
Hi @GGKF ๐, good job on completing this challenge! ๐
Here are some suggestions you might consider:
- Use the main tag instead of
<div class="container">
and wrap the cards to improve the Accessibility of the page. - In this challenge, you should not use the background property to set the image because this image has semantic meaning. Use the CSS background property if the image is not part of the content.
- You can use a <picture> tag when you need to change an image in different viewports. Using this tag will prevent the browser from loading both images, saving bandwidth and preventing you from utilizing a media query to modify the image.
Example:
<picture>
<source media="(max-width: 767px)" srcset="./images/image-product-mobile.jpg">
<img src="./images/image-product-desktop.jpg" alt="your_alt_text">
</picture>
Overall well done!
I hope those tips will help you. Good job, and happy coding!
Marked as helpful
2