Frontend Mentor - QR code component

Community feedback

• 

  Joachim•840

  @Thewatcher13Posted over 1 year ago

  Your alt text should be much more clear something like a qr code to frontendmentor Your class names too Don't skip the order of headings in html , you can't have an h2 before an h1... Add main landmark to your projects.

  note, please apply the feedback asap in your project, we give feedback for a reason. I Say this bcs I saw the feedback you get isn't applied to your project.

  Marked as helpful

  1
• 

  hitmorecode•6,680

  @hitmorecodePosted over 1 year ago

  The qr-code image is not showing, you need to change the url from this <img src="/images/image-qr-code.png" alt="qrcode" class="qrcode"> to this <img src="images/image-qr-code.png" alt="qrcode" class="qrcode">

  Also it's not good practice to have the style inside the html file. It's best to create a separated file for CSS

  Use flexbox on the body to place the card in the middle of the page

  Marked as helpful

  1

  ---

  

  UchihaHoff•10

  @UchihaHoffPosted over 1 year ago

  @hitmorecode Thank you for the insights!

  0

  

  hitmorecode•6,680

  @hitmorecodePosted over 1 year ago

  @UchihaHoff glad I could help

  0