Skip to content
Submitted over 2 years ago

CSS variables, flexbox, classes

denossart•20
@denossart
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

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

Please log in to post a comment

Log in

Community feedback

  • Reza Jaber•200
    @rezajaber

    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
  • youssef•460
    @Youssef5107

    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🔥💪

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