Order-summary-component-main

Community feedback

• 

  Daniel 🛸•44,250

  @danielmrz-devPosted 11 months ago

  Olá, Camila!

  Seu projeto ficou excelente!

  Tenho algumas sugestões, caso você queira melhorá-lo ainda mais:

  • Pra melhorar a otimização SEO e a acessibilidade do seu projeto, tenta utilizar mais as tags semânticas ao invés de div. Por exemplo, sua div.container poderia ser main.container. Isso não muda nada visualmente, mas torna o seu código HTML mais semântico.

  • Vi que você utilizou margin-top: 10%; pra afastar o card do topo da página. Existem duas formas muito eficientes pra centralizar objetos sem o uso de margin ou paddings:

  Você pode aplicar isso ao body:

  body {
      height: 100vh;
      display: flex;
      justify-content: center;
      align-items: center;
  }
  

  Ou pode aplicar isso ao elemento que você quer centralizar:

  .element {
              position: absolute;
              top: 50%;
              left: 50%;
              transform: translate(-50%, -50%);
  }
  

  Espero que essas sugestões sejam úteis pra você!

  Fora esses pequenos detalhes, seu projeto ficou show!

  Marked as helpful

  1

  ---

  

  Camila Maximo•30

  @CamilaMaximoPosted 11 months ago

  Muito obrigadaaa!
  Irei utilizar na próxima. GRATIDÃO DANIEL! @danielmrz-dev

  1