Submitted about 2 months ago
In this Responsive web page I used HTML CSS
@radhaBharadwaj
Design comparison
SolutionDesign
Community feedback
- @khatri2002Posted 20 days ago
The developed design seems good. However below are my suggestions to make it more accurate:
- Use of CSS grid layout will be a better approach here, instead of flex-box. Multiple flex-boxes in the same container can be little tricky to manage. As I can see in your design, at some specific resolutions, the gaps between cards are not consistent; which could also the reason of 2 cards with white background having more height than others.
- The background image included in card-1 appears to be huge in tablet devices (iPad Mini, iPad Air, iPad Pro), as it is covering almost entire height of the card.
- The person image resolution needs to be reduced.
- The card padding and font-size can also be reduced, in order to make an overall smaller container; which may fit in the desktop screen, preventing the scroll behaviour of page in desktop resolution.
Take the feedback very positively and keep the work going! Happy Coding!
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