Skip to content
Submitted about 2 years ago

display flex

Saif-kh•250
@saif-kh
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

  • バレンタイン 😈•64,170
    @VCaramesPosted about 2 years ago

    Congrats @saif-kh on completing this challenge!

    Card looks good!

    To clean up your code and make semantically correct, you want to the following to you code:

        <body>
      <article>
        <img src="" alt="" />
        <h1></h1>
        <p></p>
      </article>
    </body>

    Since this is a card component, its better to wrap the entire content in an <article> element .

    The <h1> is used for headings; there are six levels of headings. Heres a link with more info on it: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Heading_Elements

    For simple text, its better to use a <p> (paragraph element). Heres a link with more info on it: [https://developer.mozilla.org/en-US/docs/Web/HTML/Element/p] (https://developer.mozilla.org/en-US/docs/Web/HTML/Element/p)

    Happy Coding!

    Marked as helpful
    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