Responsive Qr code card

Community feedback

• 

  Hassia Issah•50,390

  @Hassiai

  Posted over 2 years ago

  Replace the <div class="container"></div> with the main tag and <h2></h2> with <h1></h1>

  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 for line-height value in this challenge, remove it from the body

  give .container/main padding value instead of giving .image1 a padding value, this will apply to all the content in .container/main. There is no need to give .container/main a margin value for it to be centered on the page

  Give .container-text a padding top and bottom values, instead of .c-text margin top and bottom values

  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 HAPPY CODING

  10

  Marked as helpful