@correlucas
Posted
๐พHello Victor Okeke, congratulations for your new solution!
Here's some tips for you to improve your solution:
First of all the container size is a bit smaller, changing this width you avoid it growing too much:
}
.container {
max-width: 1110px;
}
The html structure youโve used here works, but if you want to make this even better, you can replace the <div>
youโve used to keep the blocks and replace with some better html markup and semantic, for example, the main block can be wrapped with <main>
and each testimonial card with <article>
, then you can use a tag that not everyone knows, for the paragraph containing the quote
you can replace the <p>
with <blockquote>
that is tag the describe exactly its content.
Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp
To make this alignment, first of all put min-height: 100vh
to the body
to make the body display 100% of the viewport height (this makes the container align to the height size thats now 100% of the screen height) size and display: flex
e flex-direction: column
to align the child element (the container) vertically using the body as reference.
body {
min-height: 100vh;
font-size: 13px;
font-family: 'Barlow Semi Condensed', sans-serif;
background: hsl(210, 46%, 95%);
display: flex;
align-items: center;
justify-content: center;
}
โ๏ธ I hope this helps you and happy coding!
Marked as helpful