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

Product preview card with Sass

@stephanievanoverberghe

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


What are you most proud of, and what would you do differently next time?

RAS

What challenges did you encounter, and how did you overcome them?

The images were difficult but I managed to do what I wanted in the end.

What specific areas of your project would you like help with?

I would like to have help with images and image responsiveness.

Community feedback

Ujjwal 20

@guptaujjwal1128

Posted

Answering Questions

  • Yes, semantic html is there.
  • Yes, it is accessible. But interactive elements have missing focus state handling.
  • No, layout is not looking when changing screen size as widths are fixed for different screen sizes, % should be used instead of rem.
  • code well structured, it is readable and reusable.
  • Solution differ from design but not considerably.

Other Feedbacks

  • Inconsistent font size in html and body tag, why?
  • Use picture and sources tag for loading images as per screen size
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