Design comparison
SolutionDesign
Solution retrospective
Any feedback and suggestions on how I can improve my code are very welcome!
Community feedback
- @fananibananiPosted almost 2 years ago
Hi there! Congrats on your first solution, it looks great!
I'm at quite a beginner level in frontend development myself, but I did notice one thing right away: when the screen is wider than 1440px the card is no longer centered. From what I can see, this is getting thrown out of place because your container class' width is relying on rem sizing, rather than a percentage of the screen's width.
Hope this helps! :)
Marked as helpful1@haroon-rajaPosted almost 2 years ago@fananibanani Thank you for your suggestion now I changed the width to "vw" unit.
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