Design comparison
Solution retrospective
suggestion would be nice and welcome ✌
Community feedback
- @walmanjmPosted over 3 years ago
Hi, @Yashasvi2704 @MojtabaMosavi
please take a look updated solution ,based on your suggestion , am i do it right ?
0@Yashasvi2704Posted over 3 years ago@walmanjm Definitely an improvement over the last solution.
0@MojtabaMosaviPosted over 3 years ago@walmanjm nice job, exactly what I meant and you can fix the vertical centering by setting alig-items center on introductory.
0 - @MojtabaMosaviPosted over 3 years ago
Hi!, overall view of your solution looks good but there is two thing to improve in my opinion:
-
Add a little margin-bottom to your button around 780px so it doesn't overlap with social icons.
-
Center the image vertiacally.
Keep coding :)
0 -
- @Yashasvi2704Posted over 3 years ago
Hi @walmanjm,
The implementation looks good for mobile as well as big screens. You might wanna look into the area of
770px-1000px
where everything is too squeezed and starts overlapping.Good Work.!
0
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