@Lizzietrust
Submitted
There was no design for the mobile menu so I couldn't do that part, any suggestion on how to go about it please?
@C-Schubert94
@Lizzietrust
Submitted
There was no design for the mobile menu so I couldn't do that part, any suggestion on how to go about it please?
The solution looks pretty good! The contact needs to be a button not just a link (which is an easy fix). For the hamburger menu the way I went about it was I took the links and button and put them in their own container and made them flex-direction: column;
so they'd be under the logo and the hamburger menu. Now the CSS for the container needs to be display: none;
in the media query so it initially doesn't show them. After that, using JS you'll create a class that will toggle on and off for the container so it will display. What I mean by that is you'll have a class (let's say .open) that the JS will add to the HTML and that class changes the display to display: flex;
so the menu shows and goes away when you click the hamburger. I hope this helps!
@Mark-a-obrien
Submitted
I couldn't get the background svg image to be the right size. But I think that was because of screen size.
Looks pretty good! For the background I used background-size: contain; (or cover;) to cover the container (in this case the body). Both have their use cases, you'd just have to test both to see which fits better.