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

CSS variables, flexbox, classes

denossart 20

@denossart

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


  1. Happy to be part of this community
  2. I welcome tips to improve the naming of my classes

Community feedback

Reza Jaber 200

@rezajaber

Posted

Hey denossart, for the start you did really great, keep on going💻 A little tip, you used flexbox to center, but it did not worked as well as you would have liked, I guess.

Try to add a "min-height: 100vh;" to you .main-container. Then you should get a perfect centered result :)

Have a great day🔥

Marked as helpful

0

denossart 20

@denossart

Posted

Thank you, @rezajaber, for this useful tip

0
youssef 460

@Youssef5107

Posted

hello denossart ,you did great job. to improve your naming classes you could start by giving popular names ,for example:if you had one main title for the page content you cold name it "main-title".if you have one main page content you could name it "page-content".if you have one main page image you could name it "main-img" and so on. I hope this help you and by the time the class naming will be so easy and you will not take time thinking about a class name. happy coding🔥💪

0

denossart 20

@denossart

Posted

@Youssef5107

Thank you for your reply!

1
youssef 460

@Youssef5107

Posted

you are welcome❤️

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