Design comparison
Community feedback
- @VCaramesPosted almost 2 years ago
Hey there! 👋 Here are some suggestions to help improve your code:
- This challenges requires the use of two images 🎑 for different breakpoints. The
picture
element will facilitate this.
Here is an example of how it works: EXAMPLE
Syntax:
<picture> <source media="(min-width: )" srcset=""> <img src="" alt=""> </picture>
More Info:📚
https://www.w3schools.com/html/html_images_picture.asp
- 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
- Remove
width: 100%;
from thebody
, since it is already 100% by default and change theheight: 100vh;
tomin-height: 100vh;
for better responsiveness.
- For improved accessibility 📈 for your content, it is best practice to use
em
formedia-queries
. Using these unit gives users the ability to scale elements up and down, relative to a set value.
If you have any questions or need further clarification, feel free to reach out to me.
Happy Coding!🎄🎁
Marked as helpful0@bammytech1Posted almost 2 years ago@vcarames hi, can you please check my code now.. but am still having issues setting a color overlay to the images
0 - This challenges requires the use of two images 🎑 for different breakpoints. The
- @HassiaiPosted almost 2 years ago
give .img-preview a background-color of soft-violet add mix-blend-mode: multiply; and opacity: 0.8; instead of the class overlay to get the color of the image.
.img-preview{ background-color: hsl(); } img{ width: 100%; height: 100%; object-fit: cover; mix-blend-mode: multiply; opacity: 0.8; }
Marked as helpful0@bammytech1Posted almost 2 years ago@Hassiai hi, can you please check my code now.. but am still having issues setting a color overlay to the images
0@HassiaiPosted almost 2 years ago@bammytech1 Give the picture a background-color of soft-violet, change the value of mix-blend-mode to multiply.
Hope am helpful.
0@HassiaiPosted almost 2 years ago@bammytech1 replace the picture with a div in the html the and give it background-color . hopefully this will work.
Marked as helpful0
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