Design comparison
Solution retrospective
Feedback! feedback!...
Community feedback
- @MojtabaMosaviPosted over 3 years ago
Look at the following things:
1- I suggest you define you media query a at larger screen size like around 700px becuase there is no margin between you container and body.
2- You've set a fixed height on the body which creates some overflowing issues, for instance the heading in you header is not visible on mobile view.
Keep Coding :)
Marked as helpful1@b-l-u-ePosted over 3 years ago@MojtabaMosavi okay will take note on that and make some changes thanks
0 - @afrusselPosted over 3 years ago
Use a :hover color on button. Use some padding in mobile view
Marked as helpful0 - @thesohailjafriPosted over 3 years ago
-
Try adding cursor pointer on sign-up button (mouse-hover event)
-
Colors are a bit different in solution and design, there is a style guild available in the provided folder which contains the exact color (HSL color code) that is used in the design.
-
GREAT WORK, KEEP IT UP ~_~
Marked as helpful0@b-l-u-ePosted over 3 years ago@thesohailjafri will make some changes thanks for your feedback
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