Skip to content
Submitted over 2 years ago

QR code component

Victory Q•30
@vquiah
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

Solution retrospective

This challenge was completed using HTML and CSS. I had trouble with determining how many containers I wanted to use but figured something out in the end. What are everyone's thoughts?

Community feedback

  • Cagri Kizilkan•210
    @cgrkzlknPosted over 2 years ago

    Good job! Yes, 1 container is enough for this project. Also make sure to use landmark elements for accessibility. HTML5 Landmarks

    • You can change your <div class="container"> to <main class="container">
    • You can remove unnecessary Font Awesome script on your HTML.
    • You can add box shadow to container element, this tool can be useful : CSSmatic Box Shadow Generator
    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