Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Request path contains unescaped characters
Not Found
Not Found
Not Found

Submitted

Responsive QR code component using flexbox

P

@anubhab5

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

I could center an item both vertically and horizontally using flex.

What challenges did you encounter, and how did you overcome them?

Centering the element was a challenge and I overcome that using flex.

What specific areas of your project would you like help with?

The size/ dimension which was provided I feel is not exact match. If I get feedback on that it would be helpful.

Community feedback

Rupak Mukherjeeā€¢ 810

@hannibal1631

Posted

Good job Bhai, Ami suggest korbo margin gulo ektu improve hobe, ar qr-pic er overall padding lagbe jeta tomar nei. Bad baki sob okay.

Marked as helpful

0
Daniel šŸ›øā€¢ 44,230

@danielmrz-dev

Posted

Hello there!

Congrats on completing the challenge! āœ…

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 employing <h1> to <h6> tags for headings rather than using <p>.

The <p> tag is primarily intended for paragraphs. When it comes to headings, HTML offers us a range of heading tags: from <h1> to <h6>.

Here's a quick guide on how to use them:

  • 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.

It's more than just text size ā€” it's about structuring your content effectively.

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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord