Design comparison
Community feedback
- @ThiagoBRG60Posted 11 months ago
Oi 👋🏻!
O seu qr code component ta muito bom, aqui ta o meu feedback e algumas observações:
1 - No seu html, sempre adicione a tag <main> para tornar o html mais semântico, afinal o main define o conteúdo principal de um site. E eu recomendaria ao invés de usar o header, usar talvez uma <section> dentro do main.
2 - Para centralizar o card no meio da tela você pode usar o flexbox, torna tudo mais simples.
3 - Vi que você só definiu o padding bottom dentro do card, e para tirar a imagem da borda de cima você usou margin-top, você poderia aplicar o padding em todos os lados do card, assim ele já iria espaçar os elementos dentro do card, sem que você precisasse definir um margin-top depois.
4 - A única coisa que faltou aqui foi a fonte do texto que vem junto com os arquivos que o frontend mentor fornece.
Em geral, ta muito bom.
Continue aprendendo e melhorando, e feliz natal ! 🤝🏻
Marked as helpful1
Please log in to post a comment
Log in with GitHubJoin 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