Melvin Aguilar ๐ง๐ปโ๐ปโข 61,220
@MelvinAguilar
Posted
Hi @mati0165 ๐, good job completing this challenge, and welcome to the Frontend Mentor Community! ๐
It's a great solution and a good start. Here are a few suggestions I've made that you can consider in the future if you're looking to improve the solution further.
- Update the image selector to make responsive images.
img {
/* width: 16rem; */
/* max-height: 18rem; */
width: 100%;
object-fit: contain;
border-radius: 0.625rem;
}
- You could use a CSS Reset to remove browser built-in styles and reduce browser inconsistencies. Your body's default margin causes the vertical scrollbar to appear.
Popular reset style sheets:
Overall well done! I hope those tips will help you.
Good job, and happy coding!
Marked as helpful
1