Responsive Product preview card component Css flex

Community feedback

• 

  T

  Gwenaël Magnenat•1,270

  @gmagnenatPosted 5 months ago

  Hi, congrats on completing the challenge ! it looks quite close already :)

  To help you match a design you can use this perfect pixel chrome extension. You can use the snapshot present in the starter files in this extension and it will add an overlay on top of your webpage. you can move the overlay and adapt the value in the developer console, then adapt your css with the values you find.

  You should add a proper css reset at the begining of your stylesheet it will help as well deal with default browser styling conflict. good and modern css reset.

  I hope you find these recommendations useful. Let me know if you have any questions.

  happy coding !

  Marked as helpful

  2
• 

  T

  Grace•31,090

  @grace-snowPosted 5 months ago

  Some quick feedback

  • remove explicit widths. The component should only have a max width in rem.
  • fix the html. Specifically heading use and order, and use of image alt. See https://fedmentor.dev/posts/html-plan-product-preview
  • use a css reset at the start of the styles.
  • you shouldn't need any !important in the styles. That's a sign there are problems in the css. Fix them don't try and brute force override them.
  • don't guess font sizes. I can tell you're choosing random rem values. They shouldn't need to change between movie and desktop. Use the size the style guide says for body/paragraph.
  • the img needs object-fit
  • the picture element should only have one source in this for the desktop image.
  • only set media queries when there is room for the layout to change. Define media queries in rem or em not px. And make the mobile styles the default with larger screen styles in the media query. See https://fedmentor.dev/posts/responsive-meaning/

  Marked as helpful

  1

  ---

  

  tchananet•170

  @tchananetPosted 5 months ago

  Thanks very much @grace-snow

  0
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 5 months ago

  Hello there!

  Congrats on completing the challenge! ✅

  Your solution looks excelent!

  I have just one suggestion:

  📌 To improve semantic clarity, try to maintain the titles hierarchy with <h1>, <h2> <h3>, and so on.

  It's more than just text size — it's about structuring your content effectively:

  • <h1> to <h6> are used to define HTML headings, with <h1> being the most significant.

  While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.

  Hope this suggestion proves helpful! Keep up the great work!

  1