@danielmrz-dev
Posted
E aí @eric-suzuki-dev!
Belo projeto! Ficou show!
Tenho apenas 2 sugestões:
- Crie classes auxiliares pra elementos que tem o mesmo estilo. Por exemplo, vi que os seus 3 cards compartilham essas propriedades:
height: 25.625rem;
width: 13rem;
padding: 2.8125rem 2.8125rem;
Você pode criar uma classe auxiliar, colocar essas propriedades nessa nova classe e adicionar essa nova classe aos 3 cards. Dessa forma você evita repetição de código.
- A segunda sugestão é envolver os seus cards dentro da tag
main
, indicando que esse é o conteúdo principal. Além de tornar seu HTML mais semântico, você consegue resolver o problema daborder-radius
. Ao invés de adicioná-la a cada card separado, você pode adicionar a borda aomain
container. Mais uma coisa que irá reduzir seu código.
Fora esses pequenos detalhes, seu projeto ficou excelente!
Espero que ajude! 😁
@eric-suzuki-dev
Posted
Ola @danielmrz-dev, Obrigado pelo feedback