QR Code Component

Community feedback

• 

  Mohammed Hassan•240

  @MrMohammedMathPosted over 1 year ago

  Hello and congratulations about finishing your first challenge  I have some suggestions that I thought it will be helpful.

  • make sure to have at least one <main> tag. you can use it instaead of div with class container  Note: this will remove the "All page content should be contained by landmarks" warning
  • try to make .card-container take the full width and height using width: 100%; min-height: 100vh.

  Marked as helpful

  0

  ---

  

  justinrieman•30

  @justinriemanPosted over 1 year ago

  @MrMohammedMath Thank you! I think I was confusing the desktop-preview.jpg with the desktop-design.jpg which is what made me want to change the width of .card-container.

  0

  

  Mohammed Hassan•240

  @MrMohammedMathPosted over 1 year ago

  @justinrieman just focus on desktop-design.jpg Happy coding.

  0