dewslyse• 3,025
@dewslyse
Posted
Hello Mohammed! Congrats on your submission! Well done. Some things to consider:
- You could center the content to the page by adding the following to your css
body {
display: flex;
justify-content: center;
align-items: center;
flex-direction: column;
min-height: 100vh;
}
-
You could add a
max-width
(e.g: 1440px) to yourmain
to prevent content from spanning the full with of the page. The page looks better that way. -
In case you decide to take the above into consideration, you can tuck the background images nicely in the opposite corners by applying the following to the
body
in css
background: url("./images/bg-pattern-top-desktop.svg") no-repeat left top, url("./images/bg-pattern-bottom-desktop.svg") no-repeat right bottom;
- By the way, you don't need the
background-color: var(--white)
in themain
. Consider removing it.
Happy coding
Marked as helpful
0