QR Code Component | HTML |CSS

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾 Oi de novo Ághata Almeida, tudo bem? Parabéns pelo desafio!

  Sua solução ficou muito boa e eu já vi mto melhoria desde o primeiro desafio, aqui umas dicas pra vc:

  O card ficou um pouco pequeno, tenta usar um valor em torno de max-width: 320px pra fazer ele ficar responsivo e com o tamanho correto:

  .qr-container {
      background-color: white;
      /* height: 400px; */
      max-width: 320px;
  }
  

  Perceba que vc nao precisa definir altura para o container pq ele cresce com o tamanho dos elementos dentro dele e o padding, é automatico.

  Vc pode limpar o seu codigo um pouco deixando só uma div segurando tudo (img, h1 e p) as outras nao sao tao necessarias assim.

  👾Minha avaliação para sua solução: ⭐⭐⭐⭐

  👋 Espero ter ajudado e continue no foco!

  Marked as helpful

  1
• 

  Adriano•38,390

  @AdrianoEscarabotePosted over 2 years ago

  Oi Ághata, tudo bem?

  Gostei bastante do resultado do seu desafio! Tenho algumas dicas que acredito que você ira gostar:

  Percebi que você colocou uma classe no seu elemento p, aconselho você a usar classes em elementos apenas quando tiver mais de um elemento do mesmo tipo.

  O resto está ótimo! Parabéns pelo resultado.

  Espero que ajude 👍

  Marked as helpful

  1