Design comparison
Solution retrospective
How to better implement mobile responsiveness. Any improvement I can make on my CSS and ways to make it cleaner? Thanks a ton!
Community feedback
- @HassiaiPosted over 1 year ago
to center .qr-container on the page using flexbox, add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body
body{ min-height: 100vh; display: flex; align-items: center; justify-content: center; }
For a responsive content which wont a media query for this challenge,
- Give .container a fixed max-width value , a padding value for all the sides
max-width: 320px which is 20rem/em padding:16px which is 1rem/em
, a background-color of white and a border-radius value, the rest are not needed. - Give the img a max-width of 100% and a border-radius value, the rest are not needed.
Give .content-container a margin value for all the sides, text-align: center and a font-size of 15px which is 0.9375rem, this will be the font-size of both p and h1. Give p a margin-top or h1 a margin-bottom value for the space between the text.
Hope am helpful.
Well done for completing this challenge. HAPPY CODING
1@AmeerMoustafaPosted over 1 year ago@Hassiai I will give it another shot with your suggestions tomorrow, thanks a ton for the feedback!
0 - Give .container a fixed max-width value , a padding value for all the sides
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