Design comparison
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Zulamir, Congratulations on completing this challenge!
Iโve few suggestions for you that you can consider adding to your code:
1.Instead of using a long code for the IMAGE OVERLAY. You can do it using a shortcut with a few lines of code
mix-blend-mode
, that in my opinion is a better way and makes the color tone really close to the challenge design. All you need is thediv
under theimage
with thisbackground-color: hsl(277, 64%, 61%);
and applymix-blend-mode: multiply
andopacity: 80%
on theimg
orpicture
selector to activate the overlay blending the image with the color of the div. See the code bellow:img { mix-blend-mode: multiply; opacity: 80%; }
2.Improve the card's overall look by adding the rounded borders to the component and also the image using
border-radius: 15px
- Add rounded borders to the left side of the image with
border-radius: 15px 0px 0px 15px;
- Then do the same thing for the component but in the opposite borders
border-radius: 0px 15px 15px 0px;
โ๏ธ I hope this helps you and happy coding!
Marked as helpful1@zulamirsofianPosted about 2 years ago@correlucas
Hi Lucas,
Thank you very much for your feedback. I definitely will look into your feedback as I understand my code is a huge mess ๐ as I am still learning to understand the concepts. I look forward to commit some fixes and improvements in the future.
Thank you again and take care.
1 - Add rounded borders to the left side of the image with
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