Hexer• 3,660
@Phalcin
Posted
Been checking out your works. You are doing a great job man. Keep it up
Marked as helpful
1
scarydeveloper• 130
@snake321
Posted
@Phalcin thank you so much
0
Design comparison
Solution retrospective
hi guys it is my first project so plz visit my solution and share your feedback that how can I grow more and tell me my mistakes plz your feedback is most valuable for me
Community feedback
VladimirBrscic• 775
@VladimirBrscic
Posted
Hi, you do not need to set width and max-width on <body> element, <body> automatically takes up all the width of the viewport. Also by using 'grid' layout method you don't need 'margin: 0 auto' on same <body> element. Grid takes care of centering the '.container'. All in all you did good work here. Happy coding!
Marked as helpful
1
Naveen Gumaste• 10,460
@NaveenGumaste
Posted
Hay ! Good Job scarydeveloper you made it look nearly perfect to the preview Keep up the good work!
Marked as helpful
1
scarydeveloper• 130
@snake321
Posted
thanks for your feedback next time I will not do this kind of mistakes.. thank you so much
0
scarydeveloper• 130
@snake321
Posted
thanks for your feedback
0
Please log in to post a comment
Log in with GitHub
Join 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