@akshaywebster
Posted
Hi, Lance! According to what I've noticed in your code, here's my feedback:
-
First of all, in order to make the background wave image work, you'll have to put it in the body, inside of making a separate div for it. Because you've made a div for it, it's not spanning across the full screen. Also, you'll have to add a background-color as well for the bottom half of the background.
-
Second, as per the design, your card is getting too wide as the screen size increases, so maybe put a max-width for the card at a certain screen size.
-
One small suggestion I'd give is when you're using @media queries, try to specify the max-width or min-width value in px, it's easier to understand than em.
-
Also, your "Annual Plan" & prices stays on the same line after a certain screen size, because you have put them in a single element
<div class="paragraph2"><b>Annual Plan</b> $59.99/year</div>
instead of putting them in different elements like h1 & p. -
Your container is not vertically centered because you've given it absolute positioning with top=0.
Hope my feedback helps! Have a great day!
Marked as helpful