El Ghali Khalid• 2,630
@limtedtorni000
Posted
Looks great on mobile and desktop keep up the good work.
1
Krishna• 195
@krishna-nayak
Posted
Thank, @limtedtorni000
0
Design comparison
Solution retrospective
If there is anything problem or anything that can be fixed please give your suggestion.
Community feedback
Ovidiu-Antonio• 3,125
@ovidiuantonio
Posted
Hello Krishna,
You have a very good solution! It is responsive and it looks very good!
One little thing that you can modify is the with of the main component, because on a bigger screen the cards will be kinda stretched; you can replace width: 75%; (on the main component) with max-width: 75rem; (for example).
Happy coding and keep going!
1
Krishna• 195
@krishna-nayak
Posted
Thank you, @ovidiuantonio - as you told me, it is now better on a bigger screen. But when I replace width: 75%; with max-width: 75rem; It just ignores my margin and in small screen resolution sizes (width size around - 1200px), it sticks on the side of the screen. So I just added max-width: 75rem; and it works fine on a bigger screen.
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