hitmorecode• 6,230
@hitmorecode
Posted
Nice well done. Just two thing
- The background image is not filling the width of the page
- Start using
min-height
instead ofheight
body { font-family: Red Hat Display;
display: flex;
justify-content: center;
align-items: center;
margin: 0;
min-height: 100vh; /* for good practice start using min height */
width: 100%;
background-position: center top;
background-attachment: fixed;
background-repeat: no-repeat;
background-image: url(pattern-background-desktop.svg);
background-size: contain; /* add this line to fix the background image */
background-color: hsl(225, 100%, 94%);
}
Marked as helpful
1
lavendar713• 150
@lavendar713
Posted
I just did as you said and I noticed the difference, before submitting my solution I tried to achieve this result but I couldn't. Thanks for the tip.
0