Jonathan Martínez• 3,275
@Jommartinez
Posted
There is nothing wrong, the only thing, remember that the IDs must be unique and you are repeating them
3
Design comparison
Solution retrospective
Still so much to learn and fix. Any advice would be great.
Community feedback
Ovidiu-Antonio• 3,125
@ovidiuantonio
Posted
Hello,
Pretty nice solution so far! You're doing great, but there are still some things that you can do better:
- add
cursor: pointer;to all clickable elements (buttons, footer links, social links) - do the mobile version; the desktop one is looking great and I'm waiting to see the mobile one done
These are the media queries I use in every solution:
- 0-600px (mobile)
- 600-900px (tab port)
- 900-1200px (tab land)
- 1200px (large screens)
Happy coding! Keep going!
2
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