Design comparison
SolutionDesign
Solution retrospective
Don't forget to tell me if something is wrong in my code or if i can improve my code and give your opinion/feedback.
Community feedback
- @ashiqfuryPosted almost 3 years ago
Great work 👍
- After importing fonts, you should use that in
body
like:
body { font-family: 'Red Hat Display', sans-serif; }
- Don't specify the
height
property in.card
, the content will overflow according to screen sizes. Alternately, you can usemin-height: 525px
- In design, there is some background image. You can achieve this by using:
body { background: url(./images/pattern-background-desktop.svg) hsl(225, 100%, 94%) no-repeat; }
I hope my suggestions will help to improve your future projects. Thank you.
Marked as helpful0@NabilWasirPosted almost 3 years ago@ashiqfury Thanks for your feedback, appreciate it.
0 - After importing fonts, you should use that in
- @NaveenGumastePosted almost 3 years ago
Hay ! Nabil Wasir Good Job on challenge
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