@elaineleung
Posted
Hi Eliabe, welcome to Frontend Mentor, and great job working on this challenge, as it looks really good! 😊
One suggestion I have is to use height: 100%
for #image
instead of the fixed pixel height you have; the reason is, in mobile view when I resize the browser and make the width smaller than 300px (I wasn't intentionally trying to do this, more of an accident but led me to discover this), the image separates from the text, and using height: 100%
can keep that from happening here. You can also add a display: block
to your img
to remove that tiny almost 1px spacing that's underneath the image, which occurs due to img
being an inline element and thus have properties like line height. Well done on the whole!
Marked as helpful
@eliabedasilva
Posted
@elaineleung I also encountered this problem of the image separating from the text, but I had no idea how to solve it. Thanks for the guidance.