QR code component using CSS Flexbox

Community feedback

• 

  Eugenia Antonova•400

  @EugeniaAntonovaPosted 11 months ago

  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