Skip to content
Submitted over 2 years ago

flexbox

Bruno Afonso•110
@BrunoAfonsoOliveira
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Community feedback

  • Lucas 👾•104,400
    @correlucasPosted about 2 years ago

    👾Fala Bruno sou eu aqui de novo, tudo bem? Parabéns pelo desafio!

    O seu componente já está muito bom, o design está igual o do desafio, mas você poderia melhorar mais ainda reduzindo html e deixando o código mais limpo. Por example você pode segurar todo conteudo em uma div só ou no <main>. Se você notar nem precisa adicionar nenhum div além da principal pra segurar o conteudo (img, h1 e p). A estrutura ideal e mais limpa possível pra esse desafio é essa aqui:

    <body>
<main>
<img>
<h1></h1>
<p></p>
</main>
</body>

    👋 Espero ter ajudado e continue no foco!

    Marked as helpful
    0
    Bruno Afonso•110
    @BrunoAfonsoOliveiraPosted about 2 years ago

    @correlucas Eu vi em algum lugar que sempre é interessante manter uma imagem dentro de uma div, por isso mantive, o que acha disso?

    0

Please log in to post a comment

Log in with GitHub

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