Design comparison
SolutionDesign
Solution retrospective
feedback will be appreciated
Community feedback
- @artimysPosted about 4 years ago
Hi Degoke, Nice job. Very clean use of grid template areas.
Some feedback:
border-radius
andbox-shadow
on your.wrapper
container is missing- Add some line spacing with
line-height
for thebox-1
paragraph - Add some spacing between
$29 per month
and your sentence inbox-2
. - For the 'Why Us" box, use an unordered list for the content instead of a
p
tag - For this challenge, I suggest changing the max-width on your
.wrapper
container to usepx
instead ofvw
. The viewport width will really stretch it out in higher resolution monitors. Trymax-width: 635px
or 650px
Hope it helps. Keep on coding!!
1
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