Design comparison
Community feedback
- @denieldenPosted over 2 years ago
Hi Frida, good job!
for improve you code add
main
tag and wrap the card for Accessibility and useh1
for the title and noth4
. Overall you did well 😉Hope this help and happy coding!
1@FridaWaldtPosted over 2 years ago@denielden Thank you for your help Deniel, great advice on what I can change for my next project! :)
1 - Account deleted
Greetings, @FridaWaldt! 👋
Good job on this challenge! you can use
max-width
to make the card responsive instead of specifying the width of the card. You can also fix abox-shadow
to the card. You can also take a look at the mobile view.I hope this helps! Keep up the good work! 👍
1@FridaWaldtPosted over 2 years ago@munozrc Thank you so much Carlos, I really appreciate your feedback! The max-width is something I often forget, it's a great reminder! A box-shadow is a great addition I will remember for my next challenge, thank you again!
1
Please log in to post a comment
Log in with GitHubJoin 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