@mihai3636
Latest solutions
Latest comments
- @AtefZaky
Hello! congratulation on completing your first challenge 🤟
I have some suggestions to make it better and I will try to answer your questions🔽
-
in the mobile view you should give some padding to the container to make space between the card and the browser window in extra small devices. and change the
height: 100vhtomin-height: 100vhlike that when the device's height is small your content will take up its space and the scroller will appear -
in this challenge, you don't need to give the container max-width because the content doesn't take the full screen but when you make this you will need to give the background color to the body or the parent of the container to see the background on a large screen.
I hope this helps you, overall nice solution, happy coding 👨💻
Marked as helpful -
- @faressalem227@AtefZaky
Good job.., but you need more padding in the footer and the header, Increase the size of the logo in the header and footer and the size of links in the header, Make the heading in the first section and the testimonials section uppercase and you can give the first section min-height: 100vh like that when the device screen height is small your content will remain on its shape Good luck.
Marked as helpful
