Design comparison
Solution retrospective
Feedback is always welcome and helpful for me !
Update background
Community feedback
- @liampcPosted almost 4 years ago
Hello!
Here are some of my suggestions:
-
add a container div that will hold all the elements and set a max-width into it so that the elements won't spill outside of that area
-
try to use :root Selector, for custom properties like colors and fonts https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties
-
try to use css grid or flex, it will help for responsive design, search for CSS tricks for info
-
I just learned this recently here, you can use the background property and add multiple images plus the white background instead of adding the image itself in the html. https://www.w3schools.com/css/css3_backgrounds.asp
Happy coding!
2 -
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