@MelvinAguilar
Posted
Hello there 👋. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- Apply borders to the container, not individual images or components. Use
overflow: hidden
on the container to clip excess from the image for a similar effect.
- Optimize image loading using the
<picture>
tag for different versions. Example:<picture> <source media="(min-width: 460px)" srcset="./images/image-product-desktop.jpg"> <img src="./images/image-product-mobile.jpg" alt="{your alt text goes here}"> </picture>
- Use relative units like
em
orrem
for font-size instead of pixels. Resource 📘
- Prefer
min-height: 100vh
overheight
to avoid cutting off components on smaller screens.
I hope you find it useful! 😄 Above all, the solution you submitted is great!
Happy coding!
Marked as helpful