All comments

• 

  saad yasin• 20

  @SaadYasin

  Submitted about 2 years ago

  

  Responsive product preview card using flex box in css

  1

  30

  

  Aleks• 70

  @AleksAus

  Posted about 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

  @Lemirq

  Submitted about 2 years ago

  

  Responsive Product Preview Card Component using CSS Flexbox

  1

  10

  

  Aleks• 70

  @AleksAus

  Posted about 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