@Antonvasilache
Posted
Hello there! 👋
Nice work with the challenge.
A few suggestions which I think could improve the solution, if you are interested:
The header should have a bigger font to stand out from the rest. You could also adjust the font weight using the values in the style guide, like so:
.title__header {
font-size: 2.4rem;
padding-bottom: 0.2em;
}
.title__header--dark {
color: var(--VeryDarkBlue);
font-weight: 200;
}
.title__header--light {
color: var(--VeryDarkBlue);
font-weight: 600;
}
Another easy win is to add border radius to the cards:
.section__content {
margin-bottom: 1.5em;
box-shadow: 0.25em 0.25em 0.75em rgba(0, 0, 0, 0.25);
padding: 1.5em;
border-radius: 0.4rem;
}
Good job otherwise, I like the mobile responsiveness.
Hope you find this useful. Cheers!
Marked as helpful
@Biskup85
Posted
Yes, I really thank you for your suggestions @Antonvasilache, and I'm using these next time.