@correlucas
Posted
👾 Hello Kam, congratulations for your first solution here and feel welcome to the Frontend Mentor Community!
You can improve your solution doing some changes, see the tips below:
1.Add the background-color to the body instead of the container, the background-color will have the same properties of the element its inside, in your case the background-color doesn't fill the background because its inside the container.
`body {background-color: hsl(212, 45%, 89%);}
2.A better html structure you can use is one with only the necessary elements, you've add some div that are too much. See how can you build it with few elements:
<main class="container">
<img>
<h1> </h1>
<p> </p>
</main>
The rest you've done really good, I can see that the card is also responsive and this is great.
Congratulations and keep it up KAM!
Hope it helps you and happy coding!
Marked as helpful
@novatok
Posted
@correlucas Thanks for the feedback. I will put it into practice.