Skip to content
Submitted over 1 year ago

HTML, CSS, Media queries

Kamal Muhammad•220
@Meetkamal256
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

  • IryDev•1,580
    @IryDevPosted over 1 year ago

    Hey @@Meetkamal256 well done for completing this challenge 😄😄

    I have come advice in order to improve your solution :

    • Your page should have at least one h1 tag
    • I suggest replacing the <h2> tag with the h1
    • You didn't use semantic HTML that can improve accessibility on your website
    • Wrap the whole main content of your website in the semantic <main> tag

    HTML :

    <body>
<main>
...
<h1>Improve your front-end skills by building projects</h1>
...
</body>

    I hope you'll find this helpful😄

    Marked as helpful
    0
    Kamal Muhammad•220
    @Meetkamal256Posted over 1 year ago

    @IryDev I really appreciate you taking the time to check my work, i will make the necessary corrections thank you

    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