Gio Cura• 650
@GioCura
Posted
Hi! Add "vertical-align: bottom" to your ".image img" and the empty space will disappear. 🙂
Marked as helpful
1
Julia• 60
@juliavilbert
Posted
@GioCura Thank you friend! :)
0
Design comparison
Solution retrospective
Guys this is my third project and I still couldnt figure out how to eliminate that line under the image, feedback welcome! Do I need to use <main> the way i used it? How to make that main image fit the whole content? thanks :)
Community feedback
Francisco Carrillo• 5,540
@frank-itachi
Posted
Hello there 👋. Congratulation for completing the challenge👍!
I have some suggestions about your code that might interest you.
HTML 📄:
- Since the mobile design has a different image, you can use the
<picture>tag that allows you to interchange the images depending of the viewport size. Red more about this awesome tag here. - Get rid of the
<div class="container">because it’s having no effect to the descendant element. Keeping the body element rules you already have will work as well.
CSS 🎨:
- You can also use the
text-transform: uppercase;property to make the perfume word appear in upper case even though you typed it in lower case in the HTML file.
I hope you find it useful! 😄 Above all, you did a good job!
Happy <coding /> 😎!
Marked as helpful
1
Julia• 60
@juliavilbert
Posted
@frank-itachi thank you friend!!
0
Please log in to post a comment
Log in with GitHub
Join our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord