Submitted about 2 years ago

QR code component

@karpovska

A solution to the QR code component challenge

Solution retrospective

Hi guys, I'm not sure about the css files structure. Would it be better to leave as it is or make it all in one file in such small projects? Thanks in advance.

Code

Please log in to post a comment

Community feedback

Francesco Merighi•930
@francescomerighi1202
Posted about 2 years ago
Hi there!

Your solution is great!

On projects of this size, I recommend putting everything inside one file.

It makes everything easier, even for those who need to look at your solution in the future!
Marked as helpful
Bader Idris•2,880
@Bader-Idris
Posted about 2 years ago
You can set the container in the middle of the screen whatever user changes it when you add these properties to it in CSS:
```
.container {
  position: absolute;
  top:50%;
  left: 50%;
  transform: translate(-50%, -50%);
}
```
the new feature is transform, it has many lovely properties you can discover, I personally love it. Hope it's useful
Marked as helpful
Marjan•320
@MarjanZivkovic
Posted about 2 years ago
Congrats on your first project! Very well done! Any new file your page needs to load affects performance of your website. So I'd suggest putting everything into one .css file, especially for the small projects like this one.
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