QR code component

Community feedback

• 

  Abdul•8,540

  @SamadeenPosted over 2 years ago

  Hey! Cheers on completing this challenge! -Here are my suggestions

  1.You can use the margin down to push your card a little bit down to make it at the center of your page.

  2. You can always use <main> to wrap your content instead of <div> . Over all you did very well.. Happy ☺ coding!!!

  Marked as helpful

  1

  ---

  

  a coffee mug•60

  @eeffoCdecIPosted over 2 years ago

  @Samadeen thank you for your feedback 🥰

  0
• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello a coffee mug ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • Always use h1 first and then h2, h3 and so on

  • Headings should only increase by one use h1 first then h2 etc..

  • Add Main tag after body <main class="container"></main>

  happy Coding😀

  Marked as helpful

  1

  ---

  

  a coffee mug•60

  @eeffoCdecIPosted over 2 years ago

  @Crazimonk thank you for your feedback 🥰

  1
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info

  i hope this is helpful and goodluck!

  Marked as helpful

  1

  ---

  

  a coffee mug•60

  @eeffoCdecIPosted over 2 years ago

  @Old1337 Thank you for your feedback! I really appreciate it 😊

  1