Eugenia Antonova• 400
@EugeniaAntonova
Posted
Hej, Jessica! I really love your work! I would only consider to get rid of unnessesery wrapper (.text) and this pice of css
.text {
display: flex;
height: 129px;
flex-direction: column;
justify-content: center;
align-items: center;
gap: 16px;
flex-shrink: 0;
}```
hard-coding height is not a good practice, as far as i know. and you don`t really need flex to vertically align text elements - they both have display: block natively - so they take the whole line anyway and align one under another beautifully.
just give margins from top to bottom and everything lines up as you need!
wish you all the best in your career!
Marked as helpful
0
