Design comparison
SolutionDesign
Community feedback
- @VCaramesPosted about 2 years ago
Hey @FranX-21, some suggestions to improve you code:
- To properly center you content to your page, add the following to your Body Element:
body { min-height: 100vh; display: grid; place-content: center; }
-
The statistics section is a list of statistics, so it should be built using an Unordered List along with a List Items Element.
-
Having empty divs is considered bad practice. to apply the image, you instead want to use the Picture Element 🎑 in your HTML code. This element will allow you to switch between images during different breakpoints.
Syntax:
<picture> <source media="(min-width: )" srcset=""> <img src="" alt=""> </picture>
Source:
https://www.w3schools.com/html/html_images_picture.asp
https://web.dev/learn/design/picture-element/
- To get the image to look like the FEM example, you are going to want to use the Mix-Blend-Mode along with the Multiply Value and include a Opacity with the value of 0.8.
Code:
img { opacity: 0.8; mix-blend-mode: multiply; }
Happy Coding! 👻🎃
Marked as helpful1
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