3-column-preview-card-component-main

Solution retrospective
Hi everyone, any advice?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @NaveenGumaste
Hello Vanessa ! Congo 👏 on completing this challenge
Let's look at some of your issues, shall we:
-
First suggestion is to add more padding to the whole solution you did well just add padding
-
Reduce the padding in button from left and right
-
Make all title uppercase in Css or write in Uppercase in HTML
happy Coding😀
Marked as helpful -
- @denielden
Hi Vanessa, I took some time to look at your solution and you did a great job!
Tip of graphic design: With
font-family:" Big Shoulders Display ", cursive
the browser will use the Comics Sans font when it doesn't find the first font indicated (you can seen during loading) ... for the designer it's a really awful font!- I would rather replace it with a
font-family:" Big Shoulders Display ", sans-serif
much more similar to the primary font.
Overall you did well :)
Hope this help and happy coding!
Marked as helpful - I would rather replace it with a
- @jeancarlosruiz
Hola Vanessa,
Muy buen trabajo!
Solo agregando un Text-transform: uppercase a tus titulos y un poco de margen-top a los botones desde los parrafos es suficiente.
(Te escribi en español asumiendo la informacion de tu perfil. Mis disculpas)
Marked as helpful
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