Design comparison
Solution retrospective
Very welcome any feedbacks!
Community feedback
- @HassiaiPosted almost 2 years ago
To .container on the page, align-items: center: justify-content: center; instead of giving .container position: absolute and its properties. there is no need to give the body flex-direction: column;
Give .container a padding value instead of giving img margin values for the space between the img and .container. give .container height: auto or remove the height value. change the width to max-width:300px
Give img a max-width of 100% for a more responsive image. there is no need to give the img display: flex; and a height value.
Give .txt-box a padding value for all the sides. instead of padding top and bottom value.
Hope am helpful. Well done for completing this challenge. Happy coding
Marked as helpful0@ShishKirillPosted almost 2 years ago@Hassiai thanks a lot, was waiting for this kind of feedback for a while:)
0
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