Skip to content
Submitted over 2 years ago

Flexbox and max-width for the Responsiveness

Ozochi Emmanuel•150
@Chidelemmanuel
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Lucas 👾•104,340
    @correlucasPosted over 2 years ago

    👾Hello @Chidelemmanuel, congratulations for your first solution and 😎 welcome to the Frontend Mentor Coding Community!

    Here's some tips to improve your project:

    The card is a little bit big, the correct size in this case is around max-width: 320px

    You've missed the page title, you can this to the <head> -- > <title>QR Code - Front End Mentor</title>

    You've used too much divs to have the html structure, remember that you can build this with a simple approach, without using unnecessary divs, all you need is a single <main> or <div> to keep all the content inside, and nothing more. The ideal structure is the div and only the image, heading and paragraph.

    See the structure below:

    <body>
<main>
<img>
<h1></h1>
<p></p>
</main>
</body>

    👋 I hope this helps you and happy coding!

    0

Please log in to post a comment

Log in with GitHub

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