@emestabillo
Posted
Hi Vanessa, congrats on this project! I'm not getting the 'body being pushed to the left' bug, but there's a few things I noticed:
-
The nav isn't behaving as expected on mobile. Better to put the logo and hamburger at
flex
rather than block -
I also think the nav items are not ready to be laid out at 651px. The
Get an invite
CTA in particular, looks more like a div than a button. -
story__card
s can be wrapped in ana
tag. There's also two of<small class="story-(description)"></small>
in each card. -
The footer nav list seems a little off on tablet and desktop widths
-
The social media icons could use
a
tags and hover states -
There's a bit of accessibility / HTML issues to address in the report
Lastly, I suggest going mobile-first for the next one. It'll probably be easier and you'll end up with less code.
Hope this helps :-)
@vsm1996
Posted
@emestabillo Hi!! Thank you so much for all of these great suggestions. I will work on them as soon as possible!