Design comparison
Solution retrospective
Hi everyone!
If you have any suggestions/ tips, please feel free to comment.
Community feedback
- @MoodyJWPosted over 2 years ago
This is an extremely well done project! I don't see anything that stands out in the code. Couple of styling issues, though. On a laptop screen (1440), the card itself seems a little shorter than the design. Not a very big problem at all.
The other is it looks like you didn't implement the background image. Generally the easiest way to apply a background image is to add the below styles to a container element or to the body directly.
background-image: url(./image/blahblahblah); background: var(--paleBlue); background-size: cover;
You might need to adjust based on the way your HTML elements are structured, but there are a lot of options for background images so it's usually best to play around with it in the browser dev tools. The (MDN for background)[https://developer.mozilla.org/en-US/docs/Web/CSS/background] is a great source, it should break down all of the property options.
Hope this helps you out. Great job on the challenge, keep it up!
Marked as helpful1@HamnaIshaqPosted over 2 years ago@MoodyJW Thank you for taking your time to review the project and leaving helpful advice!
I will definitely look into the styling issues you mentioned and correct them.
1
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