Skip to content
Submitted 5 months ago

First batch using flexbox and basic html and css

ebrun•10
@ebrunGit
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

What specific areas of your project would you like help with?

I appreciate any feedback on any html/css focused tool or methodology that would help make this task easier to achieve

Code

Please log in to post a comment

Log in

Community feedback

  • 𝒗𝒂𝒍•70
    @vasalma

    I think your solution is quite similar to the original project, the only thing missing is the box-shadow property that should have the white background box, but overall, it's pretty good in my opinion.

    Marked as helpful
  • P
    Gabe•320
    @gabei

    Nice job!

    As far as HTML and CSS methodology are concerned, it is often best practice to keep a separate stylesheet which is then imported into the HTML file. This way there is a sort of separation of concerns.

    This is a small project, so it fits nicely there in the header, but even a couple of extra elements added to the HTML will begin to expand that CSS to the point of looking quite bloated up there. Ina larger project, having access to global styles, etc. would be easier found in a separate file.

    Marked as helpful

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