Skip to content
Submitted about 3 years ago

flexbox, sass

sass/scss
Alessio•140
@APdev88
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Did I use Sass well?

Code

Please log in to post a comment

Log in

Community feedback

  • Rafael Takano•180
    @rafaeltakano

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

    Thank you ! Very useful tips! ✌️

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