Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Nice work with this one. Looking at yous solution, I would suggest the following for you...
- Adjust the top margin in
.container
into 7rem and height to 460px - Make the margin in
.content h4
to 0 - Add
margin-bottom: 1.5rem;
in theimg
rule set - Adjust the padding in
.content p
intopadding: 0 1rem;
- Add padding to the
attribution
- Wrap the
container
with specific tag likemain
andattribution
withfooter
tag - Include description with the
alt
inimg
tag and should not be empty
Hope this helps and Keep up the good work!
Marked as helpful1@RioCantrePosted almost 3 years ago@hackingdude7 No worries. i would appreciate it if you mark it as helpful. Cheers!
0 - Adjust the top margin in
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