@jjdavenport
Posted
The basic layout of the site is correct, on the desktop layout the large areas of blank space are likely caused by min-height: 100vh. However the mobile layout is spot on. The hover effects are not quite identical to the solution with the borders on hover being 100% instead of 50% of the element width, I also have a wierd bug on 1440p where I can only see half of the h1. The page could be made more accesible with semantic html elements used on the mobile menu, I would also disable scroll when the mobile menu is active with js. The desktop site needs more work to make is responsive on the viewports its designed for.
Marked as helpful
@tortaruga
Posted
Thank you @jjdavenport, that was really helpful! For the hover effect being 100% of the width it was because I thought the solution meant it had to 'expand' from the center when you hover it and gradually underline the whole word, but I think I just made it unnecessarily more complicated 😅