Skip to content
Submitted 4 months ago

Not many CSS tools here. A bit of Flex Box.

Francois085•40
@Francois085
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

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

I assume I can reduce the CSS code to a few lines. Feel free to advise. I would be happy to work on that.

Community feedback

  • Lance•1,580
    @IzykGitPosted 4 months ago

    Good work!

    I recommend that you look into semantic html. In your code you have wrapped everything in divs and it is ok to use divs but you want to use them sparingly if possible. This is because divs have no semantic value and doesn't give the browser context to what content is inside of them.

    So instead you want to use semantic html tags such as <main>, <section>, <p>, and so on. This will improve many areas of your code as well as SEO and accessibility. Semantic HTML is very important!

    Keep up the good work!

    Marked as helpful
    0
    Francois085•40
    @Francois085Posted 4 months ago

    @IzykGit Hi Lance ! Thanks a lot for your advice. I totally forgot that while during the challenge. I will definitely keep that in mind for my next challenges ! tks again.

    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