nice job @nonamehz π just a small detail and that is to add a touch of shadow to the card
Marked as helpful
0
Jose Rodrigoβ’ 70
@nonamehz
Posted
@PitagoraStrike thanks :D
0
Design comparison
Solution retrospective
I did my best π
Community feedback
Travolgi πβ’ 31,420
@denielden
Posted
Hi Jose, I took some time to look at your solution and you did a great job!
Also add main tag and wrap the card for Accessibility
Overall you did well :)
Hope this help and happy coding!
Marked as helpful
0
Jose Rodrigoβ’ 70
@nonamehz
Posted
@denielden thanks! :D
1
Naveen Gumasteβ’ 10,460
@NaveenGumaste
Posted
Hello Jose Rodrigo ! Congo π on completing this challenge
Let's look at some of your issues, shall we:
-
you can find text
font-sizeandstylein style.guide.md so fix that -
Always use
h1first and thenh2,h3and so on,you have usedh3 -
Warp your card content around the main tag Ex: π
<body>
<main class="container">
*all you content here*
</main>
</body>
happy Codingπ
Marked as helpful
0
Jose Rodrigoβ’ 70
@nonamehz
Posted
@Crazimonk thanks for your feedback :), i'm fixing it.
1
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