Design comparison
Solution retrospective
I have not finished yet :) (it looks my page is a bit smaller than it should :D)
Community feedback
- @tesla-ambassadorPosted over 2 years ago
Hey this is really awesome work! Since you're still working on the solution, there's a few things you might want to consider;
- I think you are working on the responsiveness cause I saw that you had commented out a media query in your css code. I had to mention just in case😅
- In order to fix the small screen size, I think you should delete the max-width property on your .main-container class in your CSS as shown below:
.main_container { width: 100%; max-width: 1000px; justify-content: center; margin: 0 auto; }
So you had already declared a width of 100% and you limited it by declaring a max-width of 1000px which is relatively smaller on some computers (You might instead want to increase the max-width to 1440px or just remove the max-width property) Otherwise, I am looking forward to your complete solution. Happy coding
Marked as helpful1@SurgetinPosted over 2 years ago@tesla-ambassador Thank you so much for your help, I'm sure I'm going to finish this, but I'm suffering a little bit with this one.. 😅
0@SurgetinPosted over 2 years ago@tesla-ambassador I'm getting closer to finish this, the mobil version is almost done. Anyway I could not figure out why my sub-menu is disappear when I'm trying to hover it.
0@tesla-ambassadorPosted over 2 years ago@Surgetin Can I dm you on the slack community? I am going to look at your css code in your repo
Marked as helpful1
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