Submitted about 4 years ago
Project tracking intro component with html css js
@Dark-Lover
Design comparison
SolutionDesign
Solution retrospective
hey guys, im waiting your feedbacks ;)
Community feedback
- @ApplePieGiraffePosted about 4 years ago
Hey, Dark-Lover! 👋
Nice to see you complete another challenge! 👍
Your solution looks good and is responsive and the mobile nav works well, too! 👏
I only suggest,
- Allowing the entire hero image to be seen when the width of the screen increases (rather than being cropped off suddenly). I think you should able to do that by taking away
overflow: hidden
from thewrapper
and settingoverflow-x
to thebody
instead. Then just make sure the body's width is100vw
. 😉
Keep coding (and happy coding, too)! 😁
1@Dark-LoverPosted about 4 years agohey my friend @ApplePieGiraffe i tried to make it as it looks in the design. i will fix it ;) . thank you again.
0 - Allowing the entire hero image to be seen when the width of the screen increases (rather than being cropped off suddenly). I think you should able to do that by taking away
- @artimysPosted about 4 years ago
Well done 👍👍 Looks good, responsive and well structured.
Minor suggestion is to add some spacing between your nav and main content.
Keep on coding!!
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