Submitted 6 months ago

flexbox

mvassari73• 80

@mvassari73

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?

hello

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

hello

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

hello

Community feedback

Harsh Kumar Dwivedi• 740

@Harsh-Kumar-Dwivedi

Posted 6 months ago

Hi @mvassari73 ,

I think you can prefer to replace <div class="container"> with <main> element and <p class="title outfit-font-b">Improve your front-end skills by building projects</p> with <h1 class="title outfit-font-b">Improve your front-end skills by building projects</h1>. These will improve accessibility and make markup more semantic.

Hope it is helpful !

Nice Work !

Happy Coding !

Marked as helpful

brendaSzu• 30

@brendaSzu

Posted 6 months ago

It looks great.

Marked as helpful

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