All comments

• 

  Omoloye Fauziyat Folashade•30

  @omoloye13Submitted over 2 years ago

  Did i replicate the design well?

  

  Qr_code component challenge using custom css

  2

  30
  

  Tamale Julius•150

  @tjewliusPosted over 2 years ago

  hello Omoloye,

  The design does look nice but I was suggesting to improve on;

  1. You had used the div to contain the page content, if you could change to <main> so as to solve the accessibility issues.

  2. Vertically align your content so it's in the center of the page.

  Marked as helpful

  0
• 

  Gabriel V.•10

  @victtor-selvaSubmitted over 2 years ago

  

  First Challenge

  2

  30
  

  Tamale Julius•150

  @tjewliusPosted over 2 years ago

  nice try. I think the design would look better if you reduced on img width say to like 80% and add margin: 2rem auto; and set its display to block. this give it some space on top and also center it horizontally.

  1

  This comment has received 1 replyShow