Skip to content
Submitted over 2 years ago

flexbox, sass

Alessio•140
@APdev88
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

Did I use Sass well?

Community feedback

  • Rafael Takano•180
    @rafaeltakanoPosted over 2 years ago

    Hello there 👋

    Great job on your solution, it looks great!

    About the accessibility issues, your html should have at least one main landmark and the page should be contained by html landmarks.

    About sass, good job using sass, you used sass nestings really well. To make it shine even more, take a look at the BEM pattern design for css.

    Hope this helps ✌

    Marked as helpful
    0
  • Alessio•140
    @APdev88Posted over 2 years ago

    Thank you ! Very useful tips! ✌️

    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