Frontend Mentor responsive QR Code using HTML and CSS

Community feedback

• 

  Daniel 🛸•44,270

  @danielmrz-devPosted 9 months ago

  Fala @iBerserker89!

  Seu projeto ficou ótimo!

  Notei que você utilizou margin pra centralizar o card. Usar margens não é a melhor opção pra centralizar elementos.

  📌 Existe uma forma melhor e mais eficiente de posicionar elementos no meio da página (tanto na horizontal quanto na vertical):

  • Aplique isso ao body (Pra funcionar corretamente, não use position ou margins):

  body {
      min-height: 100vh;
      display: flex; /*  também funciona com grid  */
      justify-content: center;
      align-items: center;
  }
  

  Espero que ajude!

  Fora isso, você fez um excelente trabalho!

  Marked as helpful

  1

  ---

  

  Luciano Barros•10

  @iBerserker89Posted 9 months ago

  Opa @danielmrz-dev!

  Muito obrigado pelo feedback! Já estudei flex, mas não sabia que era uma prática ruim usar margin e position para centralizar. Vou implementar no código. Obrigado!

  1

  

  Daniel 🛸•44,270

  @danielmrz-devPosted 9 months ago

  @iBerserker89

  De nada 🙋🏽‍♂️

  1