html css

Please log in to post a comment
Log in with GitHubCommunity feedback
- @AdrianoEscarabote
Hi nazimansarii, 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 use more than one h1, this is not a good practice since it is recommended to have only one h1 per page to inform the main title of the page! so remove all h1 and put h2
Using Flexbox or Grid on the
body
to center elements ensures a more responsive and adaptive layout, fitting different screen sizes seamlessly. It avoids manual calculations and constant adjustments needed withmargin
,padding
, or absolute positioning. These techniques provide more consistent alignment and simplify the code.flexbox:
body { display: flex; justify-content: center; align-items: center; min-height: 100vh; }
grid:
body { display: grid; place-content: center; min-height: 100vh; }
The rest is amazing.
I hope this is helpful. 👍
Join 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