@MelvinAguilar
Posted
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML π·οΈ:
-
For this challenge, You had two images, one for the product on desktop and another for mobile devices. One solution could have been to implement the
<picture>
element. This element is used to provide multiple sources or versions of an image, allowing the browser to choose the most appropriate one based on the user's device and screen size.You can read more about this here π.
CSS π¨:
- Use
min-height
instead ofheight: 97vh
. Setting the height to 97vh may result in the component being cut off on smaller screens, such as a mobile phone in landscape orientation.
I hope you find it useful! π
Happy coding!
Marked as helpful
@timavidon
Posted
@MelvinAguilar Hi, I appreciate the the time you took to write me some feedback and suggestions. thank you!