@jakegodsall
Posted
Hi 👋
Great work on the project. It looks like you've really nailed the sizing for the content of the page and the dropdowns look great.
In terms of responsivity there is a couple of issues you could work on:
-
The content of the page is not centred on larger viewports. As I understand it you have a
max-width: 1440px
set on your<body>
tag. This would be better located on this outermost<main>
containing the content and then centring the<main>
within the<body>
. There is a number of ways to do so:body { display: flex; flex-direction: column; align-items: center; }
or simply:
main { margin: 0 auto; }
You just need to remember to set that
max-width
on the<main>
element to make sure it doesn't spread the full width of the parent. -
Your mobile menu switches to the desktop menu at viewport width 768px but the login and register buttons don't come into the viewport until around 1050px, meaning there is around 300px where these buttons are not visible on the screen. Just a bit of playing around with the layout should fix that.
Hope this helps 😁
Marked as helpful