@nitinpanwar08
Posted
Hi , Good job on this challenge, it is very responsive and overall looks great
Keep contributing these awesome solutions π₯ & Happy Coding π
@snake325
Posted
@nitinpanwar08 Thank for your feedback
Looking to hire developers?
Pls give me feedback because your feedback will help me to improve my coding skills
@nitinpanwar08
Posted
Hi , Good job on this challenge, it is very responsive and overall looks great
Keep contributing these awesome solutions π₯ & Happy Coding π
@snake325
Posted
@nitinpanwar08 Thank for your feedback
Hi,
This looks pretty good. I'd reduce the max width a little and let the content be 90% wide as it goes very narrow on my mobile.
A few tweaks needed on your HTML, but all very easy to fix:
I hope those are helpful tips for you (please upvote if they are to let me know :) )
@snake325
Posted
@grace-snow Thank you so much for your feedback and thanks to tell me my mistakes Your feedback is very helpful for me and it's my request to you @grace-snow when ever I share my solution pls give your feedback . again thank you so much and your this feedback is very helpful
@snake325 this looks good now. I think you're misunderstanding what the main element is for in html though. Main is to wrap whole page content after a header, not really for use like this.
@kyrylolvov
Posted
Hey! π
Looks great to me! π
The component is completed based on the design provided and is optimized for all of the resolutions. π
@snake325
Posted
@kyrylolvov Thanks for your feedback
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