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

Responsive landing page with QR Code

@Lilian4441

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 are you most proud of, and what would you do differently next time?

Definitely most proud of figuring out how to use flexbox and grid together to format everything the way I needed it (proper spacing and margins). I think next time I'd more thoroughly investigate efficient ways to solve a problem like this. I spent too much time trying to figure out how to code this without using a flexbox and grid.

What challenges did you encounter, and how did you overcome them?

Biggest challenge had to have been justifying/aligning the content properly. I didn't have a ton of knowledge beyond base HTML/CSS, and it turned out to be way easier than I thought! I originally tried just experimenting with functions I was familiar with, but then ended up looking on Google for resources I could use to show me how to properly utilize a flexbox to position the elements the way I wanted them.

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

I think efficiency of my code is the biggest one. For example, I think my CSS Stylesheet is very oversaturated, and was wondering if there were changes that could be made to my HTML and CSS files to make it less overwhelming but still have the same effect as I have now.

Community feedback

@chilldeleuze

Posted

Good job, I think!

I have a few suggestions: I saw you're not using <p>- or <h>-Tags for your text and headers. You probably should, as this gives everything more structure.

Also maybe try to look if there's a way for to separate styling of text from more general layout questions. E.g. you are currently setting the same font-family three times. You could instead do something like this, which sets the font-family for your entire document:

body {
font-family: Outfit;
}

You should also put some padding on .flex-container-text. This way you don't need to set all these margins for .qr-header and .qr-body.

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