@vanzasetia
Posted
Hi, Himanshu! 👋
Regarding the image overlay, I recommend using mix-blend-mode
and then reducing the opacity
of the img
. For your reference (and the final result) you can see @DarrickFauvel's solution.
Some feedback from me.
- Use CSS to uppercase the text. The uppercased word in the HTML might be spelled by the screen reader (spelled letter by letter).
- I recommend adding a
max-width
to themain
element to prevent the card from becoming too large on wide screen sizes. - For the image, I suggest using
picture
element so you can show the mobile version and desktop version based on the media query.
That's it! Hope you find this useful!
@F-a-u-x-L
Posted
@vanzasetia Thanks Vanza for your valuable feedback.
Yes, I just got to know that we can use background: url() and mix-blend mode to implement the image in the desired manner. I was not aware of that we can use CSS to attach the images rather that using image tag in HTML.
I will also keep in mind the other points that you have mentioned and implement it in the code and make mobile first layouts.
Thank you, your feedback was really helpful.
@vanzasetia
Posted
@F-a-u-x-L You're welcome!
If you are going to use background-image
, then I suggest using background-blend-mode
instead. 🙂
@DarrickFauvel
Posted
@vanzasetia Thanks for the mention, Vanza!😊
@vanzasetia
Posted
@DarrickFauvel No problem! I'm happy to promote your solution since mine it's not as good as yours. 😅