Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Product preview card component (Responsive)

JoshuaMeeksβ€’ 120

@JoshuaMeeks

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P
visualdennisβ€’ 8,375

@visualdenniss

Posted

Hey,

Great work there! your submission looks very close to the solution screenshot. I've noticed few issues, so i'd like to make suggestions based on that:

  • Try to avoid giving fixed height for elements/containers that contain text elements. It will cause accessibility issue, when user changes their default font-size, text will overflow and break the layout.
  • Relatedly, avoid using px instead try to use em or rem to improve accessibility. Here is a great resource on YT for that: https://www.youtube.com/watch?v=dHbYcAncAgQ
  • You can add a cursor: pointer; to your button to indicate the user that it is a clickable area.

Hope you find this feedback 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