Design comparison
SolutionDesign
Solution retrospective
I have stumbled upon this challenge and started building it , Initially I have used FlexBox for splitting container but later learnt about CSS grid and implemented it
Community feedback
- @ahmedafsaPosted 12 months ago
Hey Susmitha,
I've reviewed your code, and it's impressive! Here are some suggestions to further enhance it:
- The design currently exhibits excessive white spaces due to the using of
.details { justify-content: space-between;}
It might be beneficial to remove this and opt for{gap:}
instead. - Consider increasing the font size of the current and old prices to better catch the user's eye.
- For the button:
- use margin instead of padding to manage the space between the text and the cart Icon.
- It's preferable to encompass the entire button with
display: flex
to align the text and image precisely in the center, using gap to control the space between the cart icon and the text and using padding to control the button's height and width
.order-button { display: flex; align-items: center; justify-content: center; gap: 16px; padding: 20px 64px;}
- I observed that you centered the card vertically using
padding: 30px
which might not maintain the card at the center consistently especially when the screen area changes Here's a simple trick to ensure the element stays centered:
.container /* or body */ { min-height: 100vh; display: flex; align-items: center; justify-content: center; }
- The code appears to lack responsiveness on mobile phones. To rectify this, you can enhance it by employing the
<picture>
and<source>
elements instead of<img>
to display the product image. and use the following media query to ensure responsiveness on smaller screens:
<picture> <source srcset="images/image-product-mobile.jpg" media="(max-width: 600px)" /> <source srcset="images/image-product-desktop.jpg" media="(min-width: 601px)" /> <img src="images/image-product-desktop.jpg"/> </picture>
@media (max-width: 600px) { .your-grid-container { grid-template-columns: 1fr; } }
Apart from those minor points, your code is excellent! Best of luck for you.
Marked as helpful0 - The design currently exhibits excessive white spaces due to the using of
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