Design comparison
Solution retrospective
If anyone can take a look at this code and give me feedback I would be grateful.
Community feedback
- @JonlyJPosted almost 4 years ago
Cara, tá muito bom!
Só faltou um detalhezinho no Mobile que não ficou centralizado. Você até usou variáveis mas seria interessnte ter uma declaração mais em relação ao propósito dela por exemplo: --background-color, foreground-color, --primary-font-size. Dessa forma na hora de mudar o "tamanho dos titulos e cor"em vários lugares ser mais intuitivo
Você acabou usando o Display flex o que é uma mão na roda, por isso seu código poderia ser mais simples, não precisaria usar positions e propriedades de direcionamento, não que esteja errado mais ficou muito sobrecarregado.
Se quiser um desafio tente re-fatorar usando apenas Display flex!
1@Lusk1nhaPosted almost 4 years agoMuito obrigado pelo feedback, usuários como você ajudam muito a comunidade.
Irei re-fazer usando todas suas dicas e tentando usar apenas Display flex.
0
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