Design comparison
Solution retrospective
Any advice would be great.
Community feedback
- @fidellimPosted about 3 years ago
Hi Berie,
Great job finishing the project. Just one thing I have noticed on desktop view. You have set your main and footer max-width to 1400px. It doesn't look great for the content of the main as it goes to the left. To solve this issue, you can this to your body tag:
display: flex; justify-content: center; align-items: center; flex-direction: column;
With this, the main tag will center. Another way to solve this is just removing max-width for main tag as well as footer tag.
I hope it helps!
Marked as helpful0@fidellimPosted about 3 years ago@Berie let me know if it works! If you also have other questions, you can ask :)
1@BernardusPHPosted about 3 years ago@fidellim thank you. I did see a problem. I came across some issues when I put in your code but I did try to fix it but I have to keep the max-width in though. If that is wrong or taboo I will find another way but I tried my best to sort out the problems.
0@fidellimPosted about 3 years ago@Artosh-afr That is alright. There are many ways to solve a problem. There is no such thing as one solution only :)
1 - @rohitdubey1352Posted about 3 years ago
Design and code are good but, some suggestions from my side is that your write is designed by and coded by something is should be bottom not on top, add some margin in button at left and right as you can see in the given design and the last thing is to add margin in images in mobile devices version. That all and Great design.
Marked as helpful0@BernardusPHPosted about 3 years ago@rohitdubey1352 I think I understand but could you explain "your write is designed by and coded by something is should be bottom not on top" Do you mean I should use margin-bottom and not margin-top?
0@rohitdubey1352Posted about 3 years ago@Artosh-afr I am saying, u can see at the top "Challenged by Frontend Mentor and coded by your name" is should be bottom(Like in footer).
0@BernardusPHPosted about 3 years ago@rohitdubey1352 thx for the suggestion but personally, I like it at the top.
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