Design comparison
SolutionDesign
Solution retrospective
Tried to apply what had been suggested from the last challenge. Any comments are welcome thanks.
Community feedback
- @dusanlukic404Posted about 3 years ago
Hi Jonathan, good job! Take a look on some suggestions here:
- I think that you don't need inline CSS for h1
- Add transition to your buttons and social icons
- Reduce the size of social icons
- You don't need vertical scrollbar because site should be landing page
Ps. On mobile is everything correct, well done!
Marked as helpful0@Nathan-FrontPosted about 3 years ago@dusanlukic404 Thanks for these suggestions really helpful on my side.
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