Submitted 6 months ago

QR CODE

GlamgarOnDiscord• 20

@GlamgarOnDiscord

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

QR code component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

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

No specific comments at this time.

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

No specific comments at this time.

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

No specific comments at this time.

Community feedback

Daniel 🛸• 44,230

@danielmrz-dev

Posted 6 months ago

Hello @GlamgarOnDiscord!

Your solution looks excelent!

I have just one suggestion:

📌 To improve semantic clarity, opt for <h1> over <h2> for your main title.

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

<h1> to <h6> are used to define HTML headings, with <h1> being the most significant.
Stick to one <h1> per page for the main title, and maintain the titles hierarchy with <h1>, <h2> <h3>, and so on.

While these adjustments might not alter the visual appearance much, they significantly enhance semantic clarity, SEO optimization, and accessibility.

Hope this suggestion proves helpful! Keep up the great work!

Marked as helpful

GlamgarOnDiscord• 20

@GlamgarOnDiscord

Posted 6 months ago

@danielmrz-dev Thanks for your helpful suggesion, Have a good day!

Geo• 20

@Geox2b

Posted 6 months ago

I would still add a box-shadow to the container-box.

Please log in to post a comment

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