Design comparison
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Hi @Pulkit-s21, how are you?
I really liked the result of your project, but I have some tips that I think you will like:
1- Document should have one main landmark, you could have put all the content inside the
main
tag click hereI noticed that at higher resolutions the layout was breaking so I made some changes to the code to resolve this:
.font-League { max-width: 1320px; margin: 0 auto; }
I used
max-width
to keep the content from stretching and usedmargin
to center it.The rest is great! Hope it helps... 👍
Marked as helpful1@Pulkit-s21Posted about 2 years ago@AdrianoEscarabote Thanks a lot for the suggestions. I implemented them but made the
max width:1440px;
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