Design comparison
SolutionDesign
Solution retrospective
I am open to all crictics please kindly comment on my project.
Community feedback
- @fatlindshehuPosted over 2 years ago
Hi,
Nice job using flex to center the component, in this way you can fix vertical centering:
-
Move the
attribution div
from thecontainer
. -
Give the
container
aheight: 100vh
in order for the vertical alignment to work -
flex-direction: row
asdefault
. -
Check the fonts and colors from
style-md
provided by Frontend Mentor ( In your component they're wrong) -
The button needs text needs to be aligned with the shop icon, also the button has a hover state.
Try to make these changes, and I think you'll be all okay!
Marked as helpful0 -
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