Ivan• 2,630
@isprutfromua
Posted
Hi there. Good job! I have a few comments and suggestions:
- your card starts to break at 1337 pixels, which is not good. It would be better if you added these styles
.text-container {
/* padding: 60px 90px 0 60px; */
padding: 60px 90px;
}
.image-container {
/* height: 100%; */
}
.image-container img {
height: 100%;
object-fit: cover;
}
So the picture fills the entire space vertically and horizontally.
- It doesn't make sense to set additional classes for statistics wrappers. They are not unique, so it would be better to use a table element in this case.
<div class="companies">
...
</div>
<div class="templates">
...
</div>
I hope my feedback will be useful.
Good luck and fun coding 🤝⌨️
0
John Idenyi• 250
@JohnIdenyi
Posted
@isprutfromua thank you!
0