Design comparison
Community feedback
- @correlucasPosted about 2 years ago
👾Hello Lucas Ferreira, Congratulations on completing this challenge!
Your solution its almost done and I’ve some tips to help you to improve it:
1.I saw that for some properties you’ve used
rem
and for otherspx
. In this case, it is better to use only one kind of unit to have a better organization for your code.relative units
asrem
orem
have a better fit if you want your site more accessible between different screen sizes and devices.REM
andEM
does not just apply to font size, but to all sizes as well.2.
Keep the image responsive
. To manage the image size, you don’t need to define thewidth
andheight
together, if you do it with different values this will make the image lose proportions, to keep the image responsive and respect the container size useimg { display: block; max-width: 100%}
this way the image resize with the container whatever its size.👨💻Here's my solution for this challenge if you wants to see how I build it: https://www.frontendmentor.io/solutions/product-preview-card-vanilla-css-and-custom-hover-state-on-hero-85A1JsueD1
✌️ I hope this helps you and happy coding!
0
Please log in to post a comment
Log in with GitHubJoin 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