Design comparison
SolutionDesign
Solution retrospective
Hi It was fun to complete this challenge. While finishing up this project, I found styling the text on the desktop media query difficult. What is the best practice for doing it correctly? Any suggestions to improve this project are welcome.
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- ⚠️ Only one
source
is needed inside thepicture
element.
- The “image” in this component are purely decorative. ⚠️ Their
alt tag
should be left blank to hide them from assistive technology.
More Info:📚
https://www.w3.org/WAI/tutorials/images/
- It is best practice ✅ to use,
classes
for styling purposes, while usingids
solely for JavaScript.
- 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 to the image. Lastly, the image’s container must have the background color ofhsl(277, 64%, 61%)
.
Code:
img { opacity: 0.8; mix-blend-mode: multiply; } img-container { background : hsl(277, 64%, 61%); }
- 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 statistics at the bottom are a list ⚠️, so it should be built using an
unordered List
element.
More Info: 📚
MDN <ul>: The Unordered List element
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding! 🎆🎊🪅
Marked as helpful1 - ⚠️ Only one
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