Design comparison
Solution retrospective
yes, I know that I have many wrongs, so let me know the right path
Community feedback
- @milosshomyPosted about 2 years ago
Hello Noureddine , congratulations on this solution. Here are some advices. You can center your .container with a flexbox just set min-height: 100vh to your body. You can apply those background images with a background-image property on body and position them where you want. https://developer.mozilla.org/en-US/docs/Web/CSS/background
I hope this helps you. Keep coding!
Marked as helpful0@Noureddine-20Posted about 2 years ago@milosshomy Thank you for your help, so I do this chellange again and if you have time see what I do and tel me your opinion again.
0 - @ShayokhShorfuddinPosted about 2 years ago
Hey Noureddine! Here are some tweaking suggestions from me -
- Looks like you need some spaces between different elements. A little adjustment will help.
- The "plan" box height is a little smaller than the original design. Try increasing a little.
- Add box shadow to the button and fix the text inside the box.
Good luck and Have a nice day! 😊
Marked as helpful0@Noureddine-20Posted about 2 years ago@ShayokhShorfuddin Thank you for your help, so I do this chellange again and if you have time see what I do and tell me your opinion again. I will be happy if you do it.
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