@DavidMorgade
Posted
Hello! congrats on finishing the challenge!. Nice work with the component!
-
First of all seems that your image is not loading properly, this is because you are not defining the relative path in your background-image correctly, try with
background-image: url(./image-equilibrium.jpg);
. -
There is a bit easy way to center your component, instead of using
margin: auto auto
in the container, try giving flex to the parent element (the body), and center it, for example:
body {
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
min-height: 100vh;
}
- And my final advice would be to start using relative units such as rems instead of px, using px is usually a bad practice and should be avoid!
Hope my feedback helps you in future projects!
Marked as helpful
@Loocee
Posted
@DavidMorgade thank you. I'd work on using rem more often.