Design comparison
Solution retrospective
proud of
- finished it and turned out how as expected
- better usage of width and max-width
- feeling good with continuing with other projects on here
issue with responsive design, fixed it by doing width and max-width and not overcomplicating the design with media queries
What specific areas of your project would you like help with?css, maybe there are redundant areas? best practices for css? responsive design tips for CSS fonts.
Community feedback
- @Alex-Archer-IPosted 4 months ago
Hey!
Congrats with your first challenge =)
I suggest you to use less divs and more semantic tags =) There aren't much tags here, but at least it could be
h1
tag for main text andp
for the second.As for css you really did it well. I can only recommend you to use
min-height
insteadheight
forbody
tag - just in case if the content would be lager than the screen. Also you can omitheight: auto
of the image cos it is a default value.Hope that helps a bit. Keep doing cool stuff =)
Marked as helpful0
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