Design comparison
SolutionDesign
Community feedback
- @AdrianoEscarabotePosted about 1 month ago
Hey Tst Max, how’s it going? I was really impressed with your project’s result, though I have some advice that could be helpful:
I noticed that your page has a scrollbar to remove it, we can do the following:
.container { height: 100vh; width: 100vw; display: flex; justify-content: center; align-items: center; /* margin: 40px 0; */ }
Everything else looks great.
Hope this helps! 👍
Marked as helpful1 - @Rodrigo-PolicarpoPosted about 1 month ago
Great work @Tasin269! I noticed that you wrote the CSS code inside the HTML file. It could be more organizated if you divided the codes into two files: one for HTML and one for CSS.
1@Tasin269Posted 17 days ago@Rodrigo-Policarpo Thanks for your review, I will try to keep things more organized by separating code files
0
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