Order summary component using flex and grid
Design comparison
Solution retrospective
Hi guys, I hope you are doing very well, any tip to improve will be very well received.💞
Community feedback
- @NomiDomiPosted almost 3 years ago
Hey there,
I really like how you name your classes. They are quite general which is great if you ever end up working on a bigger project and need to replace the values in your DOM elements.
I would suggest you be a bit more careful on the responsiveness of the design. I took a look at your solution on a fairly larger screen and the hero picture stays the same, while your other elements increase. I would put a max-width on your .content class. That way even if one looks at a bigger screen all would still be in place.
Hope this helps! Keep coding! :)
1@D3press3ddPosted almost 3 years ago@NomiDomi Thank you for your feedback, im trying to improve in the responsive design but I get very complicated with media queries, appreciate your feedback💞
0
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