QR Code Component Challenge by Almo

Community feedback

• 

  ClaudioAmareno•230

  @ClaudioAmarenoPosted 12 months ago

  Hello Alexandre! Your solution looks pretty cool. I have some tips for you if you want to improve it.

  1. You can use it to center your card: body{ display: flex justify-content: center align-items: center min-height: 100vh }
  2. Using <br> tag in HTML, it's not a good practice. Try using margin-left, margin-right for it.
  3. You should add class to your <p> and <h2>, and using it in CSS instead p, or h2

  Have a nice day and happy coding!

  1

  ---

  

  Alexandre Morgado•60

  @AlmoptPosted 12 months ago

  @ClaudioAmareno

  Hi Claudio,

  I haven't learned the display property yet. I will take a look into the margin-left and right to replace the <br> tag.

  Thanks for the feedback.

  1
• 

  Daniel 🛸•44,310

  @danielmrz-devPosted 12 months ago

  Olá, Alexandre!

  Você pode adicionar uma box-shadow ao seu card.

  Fora isso, está muito bom!

  0

  ---

  

  Alexandre Morgado•60

  @AlmoptPosted 12 months ago

  Olá @danielmrz-dev

  Vou adicionar para ver como fica!

  Obrigado pela dica!

  1