All comments

• 

  saad yasin•20

  @SaadYasinSubmitted over 2 years ago

  

  Responsive product preview card using flex box in css

  1

  30
  

  Aleks•70

  @AleksAusPosted over 2 years ago

  Great work Saad! The only thing I would recommend changing is to increase the font size of your h1 and product-price to 32px. Also your paragraph's font-size should be increased and the color should be changed to 'hsl(228, 12%, 48%)' just to better match the original design. For the HTML issue that is showing just place the images in a <div> instead of <article>.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Vihaan Sharma•250

  @LemirqSubmitted over 2 years ago

  

  Responsive Product Preview Card Component using CSS Flexbox

  1

  10
  

  Aleks•70

  @AleksAusPosted over 2 years ago

  Great work Vihaan!! Everything looks good. The only thing I could find is that you haven't added 'letter-spacing:' to your paragraph tag. Also to fix the accessibility issues, use descriptive tags such as <main><, <footer> instead of <div>. You can use divs inside the <main> tag to prevent accessibility issues from arising again.

  0