All comments

• 

  mohummed sayed•170

  @MohamedFekrieSubmitted over 1 year ago

  

  Results-summary-component

  1

  10
  

  Abu-AbdiLlah-AlBukhaaree•40

  @Abu-AbdiLlah-AlBukhaareePosted over 1 year ago

  Your solution looks nice.

  0

  This comment has received 1 replyShow
• 

  Edgar Stratulat•10

  @edgarstratulatSubmitted over 1 year ago

  I had a question about centering the box to the center, i just put margin-top: 100px :(

  

  QRCode page using HTML5 and CSS3

  2

  20
  

  Abu-AbdiLlah-AlBukhaaree•40

  @Abu-AbdiLlah-AlBukhaareePosted over 1 year ago

  Hey Edgar, you did such a nice job.

  However, it will be better if you avoid setting a fixed height for your div with class container. This will make your work look nicer.

  Remove height: 500px from div.container.

  I hope this helps.

  1

  This comment has received 1 replyShow
• 

  Jeremiah Chinwe•150

  @JeremiahChinweSubmitted over 1 year ago

  

  Results Summary Component with Flexbox

  2

  10
  

  Abu-AbdiLlah-AlBukhaaree•40

  @Abu-AbdiLlah-AlBukhaareePosted over 1 year ago

  Hello Jeremiah Chinye.

  You did a great job, but there is only one issue with it.

  Just by zooming out on the page, you'll notice that the result component isn't centered.

  You can give your div with class container a min-height of 100vh. This will center it properly.

  I hope this helps.

  0