Design comparison
Community feedback
- @HassiaiPosted almost 2 years ago
in <div class= " stats"> there is no need for <br> wrap each stat in a div with two <p> p tags sample :
<div class= " stats"> <div class= "col"><p class= "numbers"></p><p></p></div> <div class= "col"><p class= "numbers"></p><p></p></div> <div class= "col"><p class= "numbers"></p><p></p></div> in the css increase the font-size of .numbers
There is no need to give .item-content a margin auto, give it a padding value for all the sides, e.g
padding: 50px
Use the colors that were given the styleguide.md found in the zip folder you downloaded.Use rem or em as unit for the padding, margin, width and preferably rem for the font-size for more on CSS units click here
For the color of the image , add a background-color .overlay instead of linear gradient, there is no need for background position,add background-blend-mode: multiply; and opacity: 0.8 to .overlay
.overlay{ background-image: url(); background-size: cover; background-color: hsl() background-blend-mode: multiply; opacity: 0.8; }
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
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