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

QR code component using CSS Flexbox

@victorzottmann

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


Is the CSS well structured? Do the units of measurement make sense?

Community feedback

@EugeniaAntonova

Posted

Hej, Victor!

I really love your work. I am a newbie too, but still have a little bit of advise for you, if you agree to take it:

this piece of css makes a little scence

.text {
  display: flex;
  flex-direction: column;
  gap: 1.1rem;
  width: 100%;
  text-align: center;
  align-self: center;
}

h3, p {
  width: 85%;
  align-self: center;
}

you could have achieved the same result with less efford, if you would have given padding: 15px; to the container and text-align: center; to the text elements. You can also get rid of unnessesery wrapper .text - it is only needed to center and give a gap to the text elements, which is achieved as i mentiioned before + margin-bottom.

Wish you a good luck in your career!

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