P
Kent O'Sullivan• 1,870
@12Kentos
Posted
Hey @mfa34,
Nice job on completing this challenge! You did great! As for the background image I have two suggestions. First you can combine them into one line of code like so.
background: url(images/bg-pattern-top.svg) right 50vw bottom 40vh no-repeat, url(images/bg-pattern-bottom.svg) 50vw 50vh no-repeat;
That way you don't need three separate lines of code like you had, this makes it a little more shorthand. However the way you did it works just fine as well. :)
Secondly I would suggest using vw, and vh like in the example I gave. (Doesn't have to be those exact measurements) This will help it be more responsive since it is based on how big/small the users screen is.
Hope that helps!
Marked as helpful
1
