Created card using CSS Grid and Flex

Community feedback

• 

  Melvin Aguilar 🧑🏻‍💻•61,140

  @MelvinAguilarPosted about 2 years ago

  Hi @caioikn 👋, good job completing this challenge, and welcome to the Frontend Mentor Community! 🎉

  I have some suggestions you might consider to improve your code:

  • Use the <main> tag to wrap all the main content in your solution instead of using <div class="container">.

  • The cart icon is for decoration purposes only, so it can be hidden from screen-readers by adding aria-hidden="true" and leaving its alt attribute empty:

  <img src="./img/icon-cart.svg"  alt aria-hidden="true">
  

  • The <div class="botao linha"> container should be a button and not a div element.

  • On mobile devices, using a media query, change the direction of the container with class "card" with flex-direction: column;

  I hope those tips will help you! 👍

  Good job, and happy coding! 😁

  Marked as helpful

  2

  ---

  

  Caio Ikena•270

  @caioiknPosted about 2 years ago

  @MelvinAguilar thanks for the tips!

  1