@MelvinAguilar
Posted
Hi @mati0165 π, good job completing this challenge, and welcome to the Frontend Mentor Community! π
Here are some suggestions you might consider:
- Try to use semantic tags in your code. It should have the main tag that groups all of your card elements:
<body>
<main class="card">
. . .
</main>
<footer>
. . .
</footer>
<body>
- Instead of using pixels in font size, use relative units of measure like
rem
orem
. The font size in absolute length units (px) does not allow users with limited vision to change the text size in some browsers. Reference. - Use
min-height: 100vh
tobody
selector, with this property you set a height and let the element grow even more if necessary. Your elements will also be vertically centered when you use this property. Also, remove themargin: 217px;
to center correctly.
I hope those tips will help you.
Good job, and happy coding!
Marked as helpful