Design comparison
SolutionDesign
Solution retrospective
Feedbacks are appreciated.
Community feedback
- @RahulKumarGautam1636Posted over 3 years ago
Hello friend, just want to indicate that the design is not looking good when we watch it on mobile in landscape mode. You can fix this otherwise you've done a very nice work.
1@riteessshhPosted over 3 years ago@RahulKumarGautam1636 Thank you for your feedback. I'll try to fix them.
0 - @RocTanweerPosted over 3 years ago
hello @Ritesh π
Nice work on this project..!!
would like to suggest minor some things here : π
- I wouldn't add a button inside a link, you could just style the link as per the design.
- The logo at the footer should also be a link, wouldn't you say?
- When I am hovering start for free and sign up button, the whole layout is displacing. I suggest remove
.onhover:hover {border : 0}
- you don't need to write transition for all the property manually, you could say
transition : all 300ms ease-in-out;
of course when the timing of all transition is same - Should the hero text be displaced when mobile menu is toggled? (I don't know about the design) but it's not looking good(personal opinion). Please check the design, I have a feeling that something is here not right
- For responsiveness, I would suggest read this.
Hope it all helps...Happy coding..! π
1@riteessshhPosted over 3 years ago@RocTanweer Thank you for your help. I'll correct them.
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