@AdrianoEscarabote
Posted
Hi Bashar Khdr, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
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 Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!
To resolve do this:
<a class="btn-primary">Read more</a>
If we see how the layout is behaving at higher resolutions, with the help of google dev tools, we will see that it is stretching a lot, to solve this we can use a max-width with the value we want the content to stop growing and to center use a margin: 0 auto;
body {
max-width: 1920px;
margin: 0 auto;
}
The rest is great!
I hope it helps... 👍
Marked as helpful
@BasharKhdr1992
Posted
@AdrianoEscarabote Thank you so much for your valuable feedback. I will use the tips to improve the solution