@vstm
Posted
Congratulations on finishing that challenge, nicely done.
I have looked at your code with regards to the grid and there are some things I think you could do.
- You rely on
margin-top: 7em;
to vertically center the boxes, this could be solved easier if you remove the margin-top and just setalign-items: center
on the.cards-container
. Thealign-items
andjustify-content
properties are helpful to know here. - Inside the grid you wouldn't (or shouldn't) rely "margin" that often, you would rather use the
gap
property to make the space smaller/larger (which you actually used)
Hope this helps as a first pointer, if you have any questions about my comment don't hesitate to ask.