@ApplePieGiraffe
Posted
Greetings, sergiobarria! 👋
Good job on this challenge! 👏 Your solution looks good and is responsive! 👍
A few things I'd like to suggest are,
- Switching to a mobile-friendly layout a little sooner in the "Latest Articles" section (so that things don't look too squished in that section right before the layout changes).
- Adding
object-fit: cover
to the images in the cards in the "Latest Articles" section to ensure that they do not get squeezed or stretched when the size of their container changes. - Making sure the background pattern in the hero section of the page doesn't cover up the heading when the screen width decreases in the desktop layout.
- The mobile menu doesn't seem to show up when the mobile menu icon is clicked, at the moment. Perhaps consider taking another look at that to see what's happening. 😉
Hope those few tips help. 🙂
Keep coding (and happy coding, too)! 😁
@sergiobarria
Posted
@ApplePieGiraffe hi!! Thanks for the feedback. I don't know what happened with the hamburger menu, it was working the first time I uploaded (maybe I broke something between changes, and didn't noticed). In any case, it works know.
I added the object-fit property to the images.
I honestly didn't put too much attention to the in between responsiveness. Maybe I went too literal on the instructions 😅😅 (been the first time practicing on the platform and all that...). As soon as I have some time will fix the breakpoints.
Again thanks for the awesome feedback.