Design comparison
Solution retrospective
Hi everyone! Here is my solution. For this project, I tried to apply more clamp() function on font-size, padding in addition to width and it seemed to work well.
I have a question about the color of the image header. I used pseudo-elements to create the color for it. Is it good practice? Is there any other way to do it?
Appreciate your feedback for me. Thank you so much.
Community feedback
- @VCaramesPosted about 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
-
The only heading in this component is the “Get insights that help your business grow” everything else will be wrapped in a Paragraph Element.
-
The image serve no other purpose than to be decorative; It adds no value. The Alt Tag should left blank and have an aria-hidden=“true” to hides it from assistive technology.
-
To get the image to look like the FEM example, you are going to want to use the
Mix-Blend-Mode
along with theMultiply
value and include aOpacity
with the value of 0.8.
Code:
img { opacity: 0.8; mix-blend-mode: multiply; }
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🍂🦃
Marked as helpful0@nt-squaredPosted about 2 years agoHi 👋, thanks for your feedback, especially about the image!
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