Ahmed Bayoumi• 6,800
@Bayoumi-dev
Posted
Hey Freivys, It looks good!...
My suggestions:
Document should have one main landmark, Contain the component with<main>.
<main>
<div class="card-container card">
//...
</div>
</main>
- I also suggest you center the component on the page with
Flexbox, by giving the parent element<main>the following properties:
main {
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
}
I hope this is helpful to you... Keep coding👍
0
Freivys Paredes• 110
@freivys
Posted
@Bayoumi-dev Thanks for the suggestions, I'll apply those changes to the code. I will continue practicing and reviewing the good programming practices that must be applied. 👍
0
