@AdrianoEscarabote
Posted
Hi ajrm16, how’s everything? I think your project turned out great! However, I have some feedback that I think might be useful:
I noticed that you used a button in which case the best option would be an a, because in my head when a person clicks on a button written Learn More
, he is not confirming a form, or something like, it will be redirected to another page, to Learn More about!
to solve this problem do this:
<a href="/">Learn More</a>
To improve project responsiveness we can do this:
@media screen and (max-width: 575px) {
.grid {
display: grid;
grid-template-columns: 1fr;
max-width: 400px;
padding: 10px 10px;
}
}
and change this:
body {
font-family: ;
background-color: var(--main-bg);
display: flex;
justify-content: center;
align-items: center;
min-height: 100vh;
}
the old height
property was defining a fixed size for the body and was hiding the content
The rest is amazing.
I hope this is helpful. 👍