Design comparison
SolutionDesign
Community feedback
- @catherineisonlinePosted almost 2 years ago
You forgot to add hover states for the buttons and linksπ
Marked as helpful0 - @coderboi559Posted almost 2 years ago
Hello, The site is good but I would fixed a few things.
- Use CSS Grid to align everything together. Make 3-columns and align them accordingly to the design. You can also use grid for the bottom news articles to separate the image and the description of the news article.
- The mobile nav close button (X) is not fixed to the page. You need to scroll up to close the mobile nav. You can fix it by using position fixed. Maybe using JavaScript, className.toggle('fixed'). Just a quick fix.
Marked as helpful0
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