QR Code Solution utilizing HTML and CSS

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

  Hi Claire , good work!

  The best element to wrap the main content in general is main tag

  The image is broken because have a slash before in the src link:

  <img alt="QR Code" src="/images/image-qr-code.png"> remove slash
  <img alt="QR Code" src="images/image-qr-code.png">
  

  Hope this help ;)

  1
• 

  Kacper Kwinta•1,405

  @kacperkwintaPosted almost 3 years ago

  Wow, this is the first time I see a <dialog> tag! I think there is no need to use this tag, you can use a simple div

  Some tips:

  • don't use px for font-size, here's why
  • the path to the image is incorrect, as you can see, it doesn't show
  • add a little box-shadow to the box, as you can see in the design

  0