Design comparison
Solution retrospective
I’ve had quite a bit of time lately, I’ve been working on it for several days, it’s definitely not perfect. However, I am very pleased to have completed it in this form. It was a pleasant challenge for me.
Community feedback
- @RioCantrePosted over 2 years ago
Hello there! Awesome work with this project. Looking at your solution, I would like to suggest the following for you…
- Clean the whitespaces in the code
- Remove unnecessary code to keep it clean
- Simplify the
padding: 1rem 2rem 1rem 2rem;
in#nav-contact
intopadding: 1rem 2rem;
- Remove
padding-top: 12%;
,padding-left: 8%;
andpadding-right: 10%;
in the.audience
rule set and simplify it intopadding: 12% 8% 10% 0;
- In the media query
max-width: 1420px
, removemargin: 0;
,margin-left: auto;
andmargin-right: auto;
intomargin: 0 auto;
. Apply the same in methodmax-width: 992px
breakpoint
Above all, the design is well implemented. Keep it up and Hope this is helpful!
Marked as helpful1 - @denieldenPosted over 2 years ago
Hi Attila, great job and congratulations on completing the challenge.
One advice for you, try to add a little
transition
on the element with hover effect.Overall you did well :)
Hope this help and happy coding!
Marked as helpful1
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