Anny Chagas
@AnnyCaroline
All comments
- Anny Chagas• 10
@AnnyCaroline
Posted
First, your repository README is very well written!
It's also great to use CSS variables for colors. It helps a lot if you need to implement theme switching or when you need to update a color.
I missed the fonts, thought. You can download it from the link at the style-guide.md and import it at the CSS using
@font-family
.Also, the card placement doesn't seem right. I can think of two ways of solving this issue:
- Using
display: flex
: define theheight
of the body as100vh
, and make it a flex container, with bothalign-items
andjustify-content
ascenter
. - Using
display: grid
: define theheight
of the body as100vh
and make it a grid container. Then, defineplace-self: center;
for your card.
Please, let me know if you need any help fixing the code. My LinkedIn is https://www.linkedin.com/in/anny-chagas/.
1 - Using