@AdrianoEscarabote
Posted
Hi Shha5, how are you? Welcome to the front-end mentor community! I really liked the result of your project, but I have some tips that I think you will enjoy:
To align some content in the center of the screen, always prefer to use display: flex;
it will make the layout more responsive!
body {
margin: 0;
padding: 0;
display: flex;
align-items: center;
flex-direction: column;
justify-content: center;
min-height: 100vh;
}
You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers may announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader. You can read more in MDN.
The rest is great!
I hope it helps... 👍
Marked as helpful
@Shha5
Posted
@AdrianoEscarabote Thank you very much for your feedback! It is very helpful, I will use your advice for my next project :)