Submitted 8 months ago

QR CODE gpt-no-Justsu

RapidSolutions3535•20

@RapidSolutions3535

Preview site View code

This is a solution for...QR code component

HTML
CSS

1newbie

Design comparison

SolutionDesign

Solution retrospective

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

Proud that it looks kinda simillar, less research and more work.

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

Html, I dont use it very often. I search a lot in the web to be able to do it.

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

Not sure yet, kinda get the feeling for it.

Community feedback

Daniel 🛸•44,250
@danielmrz-devPosted 8 months ago
Hello @RapidSolutions3535!

Your solution looks excelent!

I have just one suggestion:

📌 To improve semantic clarity, opt for <h1> over <h3> 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!
0
karlo•90
@ktorres63Posted 8 months ago
This is correct but it would be easier to work with a separate CSS file.

If you have problems with the sizes, paddings or margins, you can check the figma file there are the sizes

If you use display: flex you can use gap:<a number>px to give spaces between elements
0

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