Design comparison
SolutionDesign
Solution retrospective
Any feedback is appreciated
Community feedback
- @mdagudoPosted over 3 years ago
This looks great! Keep coding :) Try to read about Responsive Design because your container shrinks when I resize the browser window...
1 - @tedikoPosted over 3 years ago
Hello, mohamed farid! 👋
Well done! Your solution looks good, and responds well. What I can suggest is:
- Your
.card__img
background is repeating on different screen sizes. For example on 1920px width, and also on smaller like 900px. You can fix that by adding these styles to this element:
background-repeat: no-repeat; background-size: cover;
- Also, between 800px-1200px screen size your
card__data
element is sticking to paragraph. Maybe add some margin-top to it?
Good luck with that, have fun coding! 💪
1 - Your
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