@LuisJimenez19
Posted
Hello, congratulations on completing the challenge.
The idea of centering the component is very good, if you add min-heigth:100vh; justify-content:center; overflow:hidden;
to <body>
then the content will be perfectly centered.
Then this time you don't need to use this:
`@media (min-width: 955px) #content-box { max-width: 50%; }``
If you add max-width:700px
then it won't get that big, you can give it the width you want as well as the height. Otherwise everything is fine, with practice everything gets better.
By the way, it is better to use classes instead of the id if you want to give it styles, it is a convention but it is up to you what to use.
I hope my comment is helpful👋🏽
Marked as helpful
@tabbycoleman
Posted
@LuisJimenez19 thank you! This was very helpful