TailwindCss

Community feedback

• 

  Sarah•580

  @AutumnsCodePosted 5 months ago

  Well done for completing this challenge.

  I believe the width of the main container should be smaller.

  Best regards

  Marked as helpful

  1

  ---

  

  Inaolaji Ridwan•190

  @InaolajiRidwanPosted 5 months ago

  the width of the container has been rectified. Thank You

  0
• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 5 months ago

  Hey there! 🙋🏽‍♂️

  Congrats on finishing the challenge! ✅

  Your project looks awesome!

  Here's a tip that might interest you:

  📌 When you have different versions of the same image, consider using the <picture> tag.

  This will help ensure the correct image loads on the user's device, saving bandwidth and boosting performance.

  Example:

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

  Hope you find this useful!

  Great job overall!

  0

  ---

  

  Inaolaji Ridwan•190

  @InaolajiRidwanPosted 5 months ago

  @danielmrz-dev can you explain further please coz i dont understand your point

  1

  

  Daniel 🛸•44,310

  @danielmrz-devPosted 5 months ago

  @InaolajiRidwan

  On this project, you have two versions of the perfume image: One for the mobile version and one for the desktop version.

  In cases like this, when you have different versions of the same image, it's a good practice to use the <picture> tag, because it allows you to specify which version of the image should be loaded based on the size of the screen.

  You didn't do that, so your desktop version is using the mobile version of the image. This is stretching the image.

  0

  

  Inaolaji Ridwan•190

  @InaolajiRidwanPosted 5 months ago

  @danielmrz-dev noted and thank you very much

  1

  

  Daniel 🛸•44,310

  @danielmrz-devPosted 5 months ago

  @InaolajiRidwan

  If the comment was helpful, please consider marking it as helpful 😊🙋🏽‍♂️

  0