Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted over 2 years ago
Hello there! Good job in completing this challenge! Looking at your solution, I would suggest the following for you...
- Wrap the content of
container
with specific tag likemain
. For HTML structures, refer it with this one Semantics - Include description with the
alt
inimg
tags - Instead of inline style, import it on the CSS file
- Fix the width size into
width: 330px;
in the.container
. Adjust the margin intomargin: 8rem auto;
- Add
cursor: pointer;
in.aimg
rule set. Adjust margin intomargin: 6% 0;
- Add
color: hsl(215, 51%, 70%);
and adjust margin intomargin: 0 auto;
inhr
rule set - Add
border: 1px solid white;
andborder-radius: 50%;
in.three img
rule set - Modify padding in
.padding-class
intopadding: 0 6% 6% 6%;
Above all, good job with this one. Hope this helps and Keep it up!
Marked as helpful1@MohamedWagdy7Posted over 2 years ago@RioCantre Thanks a lot, I got huge benefits from your review.
0@MohamedWagdy7Posted over 2 years ago@RioCantre I hope u can help me with this "error 404" that appears in this solution and doesn't appear when I open the GitHub view link.
0@RioCantrePosted over 2 years ago@MohamedWagdy7 I deployed my projects with Netlify. I haven't tried Github pages yet.
0 - Wrap the content 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