Design comparison
SolutionDesign
Solution retrospective
Please give feedback and ideas on how to optimize and shorten my code. Thanks!
Community feedback
- @MelvinAguilarPosted over 1 year ago
Hello there ๐. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
- You can improve the structure and readability of your content by using an unordered list instead of a single paragraph and <br> to separate each line.
- Avoid using
position: absolute
to center the coponent as it may result in overflow on some screen sizes. Instead, utilize the flexbox or grid layout for centering. Get more insights on centering in CSS here here ๐.
I hope you find it useful! ๐ Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0@htmlarsPosted over 1 year ago@MelvinAguilar In order to avoid position: absolute to center the component, I added display: grid and place-content: center; to my body tag. Is this a valid solution or can this cause issues?
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