Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

03 QR Code Component Main

@eric-suzuki-dev

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


O meu 3º desafio de HTML e CSS. Agradeço por qualquer feedback.

Community feedback

@irenanrodrigues

Posted

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-suzuki-dev

Posted

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

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord