QR scanner

Community feedback

• 

  Ibrahim Oyeniyi•450

  @MtrxxoPosted 7 months ago

  Hello, if you mean Centering the card, I might be of help. Use A div to contain every thing, even the section with the White Background, then write the following CSS rule:

  div { display: flex; justify-content: center; align-items: center; height: 100vh; }

  Marked as helpful

  0
• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 7 months ago

  Hey @kishanup78910!

  Your solution is really impressive!

  I've got a couple of ideas (about how to use HTML better) that could make it even stronger:

  📌 First: Think about using <main> to wrap your main content instead of <div>.

  Imagine <div> and <span> in HTML as basic containers. They're good for holding stuff, but they don't tell us much about what's inside or its purpose on the webpage.

  📌 Second: Consider using <h1> for your main title instead of <h2>.

  It's more than just text size — it's about structuring your content effectively:

  • The <h1> to <h6> tags are used to define HTML headings.
  • <h1> is for the most important heading.
  • <h6> is for the least important heading.
  • Stick to just one <h1> per page – it should be the main title for the whole page.
  • And don't skip heading levels – start with <h1>, then use <h2>, and so on.

  These tweaks might not change how your page looks, but they'll make your HTML code clearer and help with SEO and accessibility.

  Hope that's helpful!

  Keep up the great work!

  0

  ---

  

  Kishan Upadhyay•10

  @kishanup78910Posted 7 months ago

  @danielmrz-dev yeah sure i will implement this in my code

  0