@hitmorecode
Posted
Nice well done. Just a few tips
- If you have an element with a class or id, when applying css you can target the class or id of that element directly.
.section1 .result-title
you can remove.section1
and just target.result-title
directly. What you are doing is handy when you have elements without a class or id for example:
<section class="section1">
<p>something</p>
</section>
<section class="section2">
<p>something</p>
</section>
If you want to target only the <p>
inside .section1
then you can do it like this .section1 p {}
-
If you add
min-height: 100vh;
on the body and apply flexbox, you can remove the margin on the container. Flexbox will place the card in the middle of the page. This is much better for responsiveness of the page. -
When structuring your html, try to make a good habit of doing like this
<body>
<main>
/* everything goes in here */
</main>
</body>
Marked as helpful