@AdrianoEscarabote
Posted
Hi pandelis lagos, 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:
As you know this page is just a component, so it doesn't even need an h1
! because we don't know how important the other components of the site will be! but it's always good to prevent accessibility errors so I think it would be good for you to add an h1
in this component, besides being a good practice for when you are developing larger sites, don't worry forget about h1
.
to solve this problem do this:
<h1>Shift the overall look and feel by adding these wonderful touches to furniture in your home</h1>
To align some content in the center of the screen, always prefer to use display: flex;
it will make the layout more responsive!
Example:
body {
margin: 0;
padding: 0;
display: flex;
align-items: center;
flex-direction: column;
justify-content: center;
min-height: 100vh;
}
The rest is great!
I hope it helps... 👍
Marked as helpful