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
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive landing page using CSS Flexbox

Alexβ€’ 20

@GoodAlex223

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


Thanks for checking out my solution. If you have anything on your mind, please feel free to comment. Thanks to frontendmentor platform for this opportunity.

Community feedback

Eileen dangeloβ€’ 1,600

@Eileenpk

Posted

Hi Alex, congrats on your first Frontend Mentor project!!!! πŸŽ‰πŸŽ‰πŸŽ‰ Your project looks great and very clean. I did notice something small when I was looking at your HTML and I thought it might be helpful to point it out. Your <main class='container'></main> is this outer wrapper correct? You then have a <div class='card'></div> that is inside that main tag. When you closed off <main>, the closing tag of the </div> should be inside the </main> tag for proper nesting and readability. This will also help prevent bugs. It should look like this-

<main class='container'>
    <div class='card'>
</div>
</main>

Hope this helps! If you found this helpful please mark it as such :)

Marked as helpful

0

Alexβ€’ 20

@GoodAlex223

Posted

Hello Eileen, thanks for your comment! I really closed the tag in the wrong place, overlooked) Thank you again, next time I will be more careful!

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