@MelvinAguilar
Posted
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
-
Instead of using two empty divs and "position: absolute" to position your images, use the body element to place the two background images using a single background-image property, and remove the two divs. You will use a lot less code lines using this technique.
- Note: The comma in the background-image property separates multiple background images that are being applied to the same element.
Here's an example:
body { /* Your another styles */ background-color: hsl(185, 75%, 39%); background-image: url(../images/bg-pattern-top.svg), url(../images/bg-pattern-bottom.svg); background-repeat: no-repeat, no-repeat; background-position: right 52vw bottom 35vh, left 48vw top 52vh; } .background { /* position: fixed; */ /* top: 0; */ /* left: 0; */ /* background-color: hsl(185, 75%, 39%); */ /* width: 100vw; */ /* height: 100vh; */ /* z-index: -2; */ /* margin: 0; */ } .background-circle-top { /* position: absolute; */ /* top: 0; */ /* etc . . . */ }
I hope you find it useful! π
Happy coding!
Marked as helpful