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
Request path contains unescaped characters

Submitted

CHANEL Card

DDOL-dol• 40

@DDOL-dol

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?

Responsive website design

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

Media query for mobile and desktop

Community feedback

P
Steven Stroud• 4,140

@Stroudy

Posted

Great solution, Should be proud of this, few things i noticed,

  • align-items: stretch; is making your card do some weird stuff, you should instead use something like object-fit: cover; width: 100%; height: 100%;,
  • Should always use max-width: 100% or min-width: 100% over width:100% this will make your site more responsive check out responsive-meaning by a frontend mentor dev.
  • Your media queries should use rem or em not px.
  • <img> tag doesn't half a alt="" for accessibility and SEO,
  • Missing a <meta> description tag
  • No <main> tag
  • You have used a lot of <div> tags which is semantically incorrect for example <img> for images, <p> for paragraphs, <h1> <h2> for header tags

Keep coding, i look forward to seeing more from you.

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