Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive product-preview-card-component

@MoneebAliKarrar

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


I'll be happy to have feedback on my code structure.

Community feedback

Daniel πŸ›Έβ€’ 44,230

@danielmrz-dev

Posted

Hello @MoneebAliKarrar!

Your project looks great!

I have some suggestions about your code that might interest you.

πŸ“Œ You don't need to use words like "image", "photo", or "picture", because the image tag already conveys that information.

πŸ“Œ Use the <picture> tag when you have different versions of the same image πŸ–Ό. Using the <picture> tag will help you to load the correct image for the user's device saving bandwidth and improving performance. Example:

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

I hope it helps!

Other nthan that, great job!

Marked as helpful

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord