03 QR Code Component Main

Community feedback

• 

  Renan Rodrigues•960

  @irenanrodriguesPosted 12 months ago

  Fala Eric, tudo bem? Parabéns por concluir o desafio, ficou muito bom. Tenho algumas dicas, se você aceitar.

  Para esse desafio, na minha opinião não precisa usar muitas divs. Você poderia estruturar da seguinte forma?

  <main>
    <img>
    <h1></h1>
    <p></p>
  </main>
  <footer class="attribution"></footer>
  

  Nas classes .image e description, você usou flexbox, mas se for no DevTools, desativar ele, vai perceber que não estão fazendo efeito. Então recomendaria remover.

  Na imagem você pode deixar ela com width: 100%, assim ela vai se adaptar bem com seu .container.

  .container, você pode usar um padding: 20px; já vai adicionar para todos os elementos do card.

  Recomendo dar uma assistida neste vídeo , sobre hierarquia de títulos.

  0

  ---

  

  Eric Yuji Suzuki•270

  @eric-suzuki-devPosted 12 months ago

  Olá, sobre as dicas, eu irei já colocar em pratica nos proximos desafios. Obrigado pelo feedback.

  0