Design comparison
SolutionDesign
Community feedback
- @AbimzzPosted 4 months ago
First of, nice work on the project.. there are a few minor things i would like to point out:
- Font: the font-family used was no the required font for this project which was (outfit) and the font-weights needs to be adjusted.
- the .container should have take up 100% of the view screen as it was acting as the background for the entire website.
- paddings should have mainly been used instead of margins.
- there was no box-shadow on the card.
Most of the other issues with the site can be fix by using flexbox, so i recommend that you look into flexbox.. this should help you start https://css-tricks.com/snippets/css/a-guide-to-flexbox/
keep coding, keep improving.
Marked as helpful0@BichesqPosted 4 months ago@Abimzz wow thanks so much for this feedback. It so detailed and super helpful. I will get to looking into flexbox right away. Thanks again.
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