Guilherme• 40
@guilhermepatriarca
Posted
Great work :)
These are my suggestions for improvement:
- Use semantic HTML to replace
divwitharticlefor example - Use
altin tag<img>
Marked as helpful
0
Design comparison
Solution retrospective
What are you most proud of, and what would you do differently next time?
I don't know tbh :) Maybe i would use tailwind CSS
What challenges did you encounter, and how did you overcome them?
only with Figma, but i finally found out what fonts are there
What specific areas of your project would you like help with?
in this project i dont know :)
Community feedback
Alex Blue• 70
@fake-alex-blue
Posted
I like the approach you took to this solution. It's clear you've made some different stylistic choices, using Montserrat for the h2 text, for example - which I can't argue with personally; I think it looks good.
The only small thing I'd mention, is that you've not added the box-shadow on the card, which is a shame; I reckon it elevates the design of the original; although to be honest it's so subtle that I completely missed it on my first attempt at the challenge.
Marked as helpful
0
Please log in to post a comment
Log in with GitHub
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