Product Preview Card

Community feedback

• 

  Chirag•400

  @chirag-bishnoiPosted 8 months ago

  That float:right; is the culprit, remove that and that width:360px too! They're messing up the layout. If you want to divide the section between the articles equally the just use grid, like this:

  section{
  display:grid;
  grid-template-columns: 1fr 1fr;
  }
  

  and that will do the job 👍

  Marked as helpful

  1

  ---

  

  Eríck•130

  @Erick-SLPosted 8 months ago

  @chirag-bishnoi Thanks, I was learning better about the grid... I'll keep trying... thanks for the great tip!

  1
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 8 months ago

  Hello @Erick-SL!

  Your project looks great!

  I have a suggestion about that might interest you:

  📌 You can use the <picture> tag when you have different versions of the same image.

  Using the <picture> tag will help load the correct image to the user's device, saving bandwidth and improving performance.

  Example:

  <picture>
      <source media="(max-width: 460px)" srcset="{desktop image path here}">
      <img src="{mobile image path here}" alt="{alternative text here}">
  </picture>
  

  I hope this helps!

  Other than that, excellent work!

  Marked as helpful

  1

  ---

  

  Eríck•130

  @Erick-SLPosted 8 months ago

  Thanks for the tip @danielmrz-dev

  1