Qr_code component challenge using custom css

Community feedback

• 

  TripleA•270

  @TRIPLE-ADEPosted over 2 years ago

  Hello @omoloye, Good Job the design looks nice. Anyway, I think there are some aspects that you can improve on .

  to make your main section appear in middle
  body{ display : flex; flex-direction : column; justify-content : center; align-item : center;} and also add a min-height : 100 vh; to the body section, that will make the body section to appear in full page

  I hope this feedback would be useful)

  Marked as helpful

  0
• 

  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