@kaamiik
Posted
Hi. Congratulation for doing this challenge. I wanna mention some notes:
-
The
header
is mostly use for logo and navigation items. Here I suggest you put everything except the logo image inside themain
. -
For your buttons, You can simply wrap them in a div and use flexbox with wrap capability. I think It's more clear and clean than using
margin-right
If you add it for alignment. -
Your
h1
and it's text needs a max-width for tablet size. Also the grid images group need need to be on 1 row for tablet as I see in the design. -
And If you notice on desktop you have overflow on the x and you can scroll horizontally and in your footer if you scroll to the right, you can see the background changing. I think this challenge for the top images, need a negative margin and also
overflow: hidden;
for x axis. -
Also the blue green section in the bottom seems a footer to me. and can be outside your main as a footer element.