Design comparison
Solution retrospective
stats preview components
Community feedback
- @pikapikamartPosted over 3 years ago
Hey, great work on this one. Layout in desktop is good, just missing the color overlay. The mobile view however is not and issue in resizing/responsiveness.
Suggestions would be that:
-
On the
body
tag, you may NOT want to usewidth: 80%
, use that in thecard
selector. Thebody
should always get the whole viewport (as far as I know), also addingjustify-content: center
on yourcontainer
selector will make the card centered properly. -
Adding more
padding
to theblock
selector and change thetext-align: center
totext-align: left
or just remove thetext-align
. -
To get that purplish-effect, since you are not using any
background-image
, you can just add abackground-color: rgb(170, 92, 219)
on thecart-img-part
selector and the image inside it, you may want to addopacity
and tweak it until getting the desired color. You can also change thebackground-color
that I mentioned to be more darker. -
Lastly, on your part, make sure to make the layout is responsive. If you inspect it in dev tools, you will see that your layout is getting hidden by the browser. Make it responsive. Also do not just wait for the
375px
mobile, it is just design, not breakpoint. Adjusting it to a higher breakpoint will be really good.
Goodluck on the responsive part and still, good job on this one^
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