Jorge A. Mendoza II• 720
@JorgeAMendoza
Posted
Hello! Great attempt, the page is mostly responsive and your solution matches the design.
Some things I believe that can be improved upon:
- Starting on screen-width 425px, your mobile menu "hamburger icon" is rendering along with the desktop navigation list, causing an awkward layout. I would look back at your media queries and ensure that elements are hidden away properly.
- If a user opens up the mobile menu and expands the screen size, the "close-menu" icon appears next to the logo past the screen width of 425px. I would recommend implementing an event listener for window resizes that ensures to properly remove/hide the mobile nav menu on larger screens.
Please feel free to checkout my solution to this project for reference!
Marked as helpful
1
Kinga Miśkiewicz• 160
@codingbeary
Posted
Thank you so much! :) @JorgeAMendoza
0