Design comparison
SolutionDesign
Community feedback
- @RioCantrePosted almost 3 years ago
Hello there! Nice work with this challenge. Looking at your solution, I would suggest the following for you...
- Instead of using
section
, wrap the whole contentcard
with specific tag likemain
- Include description with the
alt
inimg
tag - Clean the whitespaces in the code
- Remove unused code to keep it clean
- Adjust the width of the
.divider
into 330px - Add
border: 1px solid white;
andborder-radius: 50%;
in the.avatar-img
rule set
Overall, you did great with the design. Hope this helps and Keep it up!
Marked as helpful1@dsaglam94Posted almost 3 years ago@RioCantre Hey there! Thank you very much for your informative reply. I really appreciate it! I checked the parts you mentioned and fixed them. It turned out better.
1@RioCantrePosted almost 3 years ago@dsaglam94 Awesome! I'm glad it was helpful. Cheers!
1 - Instead of using
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