Design comparison
Solution retrospective
I tried my best but couldn't get a hold of how to make it responsive. Can you guys direct me to some resources for learning it?
Community feedback
- Account deleted
Hello there! π
Congratulations on finishing your challenge! π
I have some feedback on this solution:
-
its probably because the container too small for the text on small screens you can use @media queries and change the width of the container to 80% .
-
Always Use Semantic HTML instead of
div
like<main>
<header>
, etc for more info. -
Consider using the h1 element as a top-level heading only (all h1 elements are treated as top-level headings by many screen readers and other tools) like for example use h1 first then h2 etc.
-
Images must have alternate text
if my solution has helped you do not forget to mark this as helpful!
Marked as helpful1 -
- @MubeenAhmadShaikhPosted almost 3 years ago
Looks good I have following feedbacks for you,
- you can use flexbox instead of margin:auto for aligning the card in center. You can refer this Flexbox guide
- And for responsive site you can refer Beginner's guide to media queries
If you get stuck checkout my solution. Hope this helps you
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