@alvarozama
Posted
Your page looks good when loaded on either desktop or mobile screen sizes. However, transitioning from one into the other makes things go a little crazy before falling into place. Some things I'd like to point out, mainly pertaining to styling, are the following:
- For some reason, the lines separating the articles in the New section overlap with other elements when changing the viewport size.
- The size of the img elements on the bottom section is different for each image. You could just give them the same fixed width and it would look much more uniform.
- Your mobile menu could use some padding since al <li> elements on the nav are too close to the edge and it looks weird.
Other than that, everything seems well done. Your approach to the burger menu is similar to mine. I, however, didn't add an overlay element and intead just 'faked' the overlay by adding a huge box-shadow to the mobile menu. Feel free to check it out. It's not perfect, but for the purposes of this challenge, I think it's a more simple way to get similar results.
Anyway, you did a very good job. Keep it up!
@Caelus111
Posted
@alvarozama thank you for these notes, I will definitely redo some things with latter projects when I get used to making landing page style websites... about the mobile menu, I watched some tutorials and I kind of learned the huge shadow method by accident 😭. anyway thanks again for the constructed feedback!!
