Hi Mukwende, you did okay. I have some suggestions though.
- Change the
div
withclass container
tomain
, as every html page must have one main tag. - Remove one
h1
heading from your page. I suggest you remove this one with<h1 class="center">Get early access today</h1>
as they ought to be just one main heading. you can useh3
and style it the way you want. - You are right, the responsiveness is not quite there. I use
media queries
for responsiveness. You used@media screen and (min-width: 768px)
. What this means is that the code below it is for screens with size 768px and above, that is for ipads and desktop. That's by the way side. Although, the responsiveness is not the problem, there's atext overflow
from :<h1 class="center">All your files in one secure location, accessible anywhere.</h1>
And<p>It only takes a minute to sign up and our free starter tier is extremely generous. If you have any questions, our support team would be happy to help you.
I suggest you adjust that. - Then set
margin:0
for the body, so that there's no visible spacing around the page.
Hope this helps, do well to update your solution when you make these changes.
Marked as helpful