@James-alderson
Posted
Hi Bruno Ginél
I have created a pull request in your repository, you can review the changes and add the changes to your design.
Marked as helpful
Looking to hire developers?
@James-alderson
Posted
Hi Bruno Ginél
I have created a pull request in your repository, you can review the changes and add the changes to your design.
Marked as helpful
@correlucas
Posted
👾Fala Bruno, tudo bem? Parabéns pelo desafio!
Acabei de dar uma olhada na sua solução e te posso dizer que os componentes não estão responsivos porque você usou várias vezes width
o melhor jeito seria você substituir todas essas propriedades com max-width
pra deixá-lo responsivo permitindo o conteúdo se ajustar junto com o container.
Agora pra ter o mesmo efeito de overlay roxo na foto, basta você colocar mix-blend-mode: multiply;
no seletor da imagem pra fazer o mesmo efeito.
Se você quiser ver a minha solução pra entender como eu montei a estrutura aqui está o link:
https://www.frontendmentor.io/solutions/stats-preview-card-vanilla-css-custom-design-text-animation-RfGBFlMpwK
👋 Espero ter ajudado e continue no foco!
Marked as helpful
@James-alderson
Posted
Hi Bruno Ginél, I saw your design, It is a good design, but there are a few issues that you can fix to help improve your design.
As follows:
These are tips to help improve your design, you can see all the changes in the commits section.
By the way, I have created a fork in your repository, you can add all the changes to your challenge.
If you are just starting to learn design, no problem. I also faced these kinds of problems at the beginning of my work. My suggestion is that before you start the new challenge, watch of the tutorial videos I'll link to below, as this will help you improve your design.
Do not forget to practice, because the more you practice, the better and faster you will design.
Happy coding
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