Design comparison
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hi Trung_TLB, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:
Add a padding to the body, so that the content in lower resolutions doesn't hit the edge of the screen!
As you know this project is based only on one page component, so no h1 is needed! because we don't know if this page will have a more important component, 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 abouth1
.<h1 class="pro-name">Equilibrium #3429</h1>
The rest is great!
I hope it helps... 👍
Marked as helpful1@TLBTrung-222Posted almost 2 years ago@AdrianoEscarabote thank you, sir. I will take these tips in a later project
1
Please log in to post a comment
Log in with GitHubJoin our Discord community
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