@emestabillo
Posted
Hey Carlos, looks good! Here are a few points:
-
You typically would need only one set of top nav instead of two. Adjust the styling of the menu to adapt to your breakpoints rather than duplicating html.
-
Instead of using divs, the links on the nav and footer can be list items inside
ul
tags, like ul > li > a. -
The text inside
.photo-design-container
should be on the bottom part of the div like on the screenshot, but for some reason the live link has the text in the opposite direction. Not sure if you pushed code last-minute butjustify-content: flex-end
should sort it.
Hope this helps!
Marked as helpful