QR Code Component Main

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  To center a content on a page, add min-height:100vh; display:flex; align-items-center: justify-content: center; to the body. There is no need to give main a margin value.

  Use rem or em as unit for the padding , margin,width and preferably rem for the font-size for more on this watch this https://youtu.be/N5wpD9Ov_To

  Hope am helpful.

  Well done for completing this challenge, you did a good job. Happy Coding

  Marked as helpful

  0

  ---

  

  Lawrence•180

  @lwrncgmbnPosted almost 2 years ago

  @Hassiai Thanks for the information, I really need that <3

  0
• 

  👾 Ekaterine Mitagvaria 👾•7,600

  @catherineisonlinePosted almost 2 years ago

  Looks great, make sure to add alt tag to the image and use h1 for the text "Improve your front-end skills by building projects "

  1

  ---

  

  Lawrence•180

  @lwrncgmbnPosted almost 2 years ago

  @catherineisonline Thank you

  0
• 

  Atharva Rane•10

  @AtharanePosted almost 2 years ago

  To start with you can center the card vertically with Flexbox, overall the project looks promising.

  0

  ---

  

  Lawrence•180

  @lwrncgmbnPosted almost 2 years ago

  @Atharane Thanks a lot ^_^

  0
• 

  Davide Di Francesco•200

  @davdifrPosted almost 2 years ago

  The work is well done and is very similar to the preview, congratulations! The only thing I would suggest adding is a slight shadow to the card, which is present in the example template. I am confident that with your skills it will not be difficult to complete this step as well! Best regards.

  0

  ---

  

  Lawrence•180

  @lwrncgmbnPosted almost 2 years ago

  @davdifr Thank you very much ^_^

  0