Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

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

@Francois085

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@IzykGit

Posted

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

Posted

@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
Discord logo

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