Design comparison
Community feedback
- @saifion33Posted about 2 years ago
Hi sheema.
Your solution is nice. If you want to improve here is some tips.
-
Wrap
<div class="container">
main tag<main><div class="container">your code</div>
-
Use
Overpass
font family that is mention in style-guide.md. for<h1>
use font-weight 700 and for<p>
use 400 font- weight -
In style.css instead of
.container ul li button
use.button
directly and addcursor:pointer
property
I hope you find this helpful.
Happy Coding
Marked as helpful0 -
- @snhodadePosted about 2 years ago
Hi! I would appreciate it if you could review my work and give feedback. Your feedback is very important to me, thanks.
0@snhodadePosted about 2 years ago@snhodade Thank you for your valuable suggestions. Will apply these changes for sure
0
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