Miran Legin• 620
@miranlegin
Posted
Hi Lenez,
congratulations on completing this challenge! You did a great job here and there are only couple of tweaks to make it even better in my opinion.
- remove the height on
main
element, you will see that thetext__content
is no longer nicely spaced vertically so you can fix it by addingpadding
to top and bottom sides odtext__content
- you can replace the
product__image
div withpicture
element and put inside both images for mobile and desktop versions accordingly. there is an example on my channel regarding this technique and you can learn more information on this article Picture element with art direction - as you already have
border-radius
declared onmain
tag, you don't need to declare it once again onproduct-image
, just addoverflow:hidden
to hide anything that might stick outside of it
Keep coding!
1