cryptosossoβ’ 150
@cryptososso
Posted
Hi, @MelvinAguilar Thank you very much for your feedback and the resources. I'll use them for my other challenges.
1
@cryptososso
Posted
Hi, @MelvinAguilar Thank you very much for your feedback and the resources. I'll use them for my other challenges.
@MelvinAguilar
Posted
Hi @cryptososso π, good job on completing this challenge! π
I have some suggestions you might consider to improve your code:
<main>
tag to wrap all the main content in your solution instead of using <div class="container">
, also, there should only be one man tag.rem
or em
. The font size in absolute length units (px) does not allow users with limited vision to change the text size in some browsers. Reference.Using flexbox layout:
body {
min-height: 100vh;
display: flex;
flex-direction: column;
justify-content: center;
align-items: center;
}
Using grid layout:
body {
min-height: 100vh;
display: grid;
place-content: center;
}
Additionally, remove the margin to center the component correctly.
.container {
/* margin: 0 auto; */
/* margin-top: 100px; */
/* margin-bottom: 15px; */
. . .
}
Links with more information:
I hope those tips will help you! π
Good job, and happy coding! π
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord