Design comparison
SolutionDesign
Solution retrospective
Your feedbacks are welcome :D
Peace โฎ๐โ
Community feedback
- @NaveenGumastePosted over 2 years ago
Hello Jay Dee ! Congo ๐ on completing this challenge
Just two suggestion :
-
Increase the size of the font so the cards will be stretched
-
Always use
h1
first and thenh2
,h3
and so on
Overall you did good
happy Coding๐
Marked as helpful0@JayDeeZzzPosted over 2 years ago@Crazimonk Thanx for the suggestions buddy ๐ Just fixed those errors. Cheerrzzzzzzzzzzzzz ๐ฅณ
1 -
- @tea-scriptsPosted over 2 years ago
Good job man.
- One little thing, you seem to have missed the borders around some images and also you could use some max width on the cards for the mobile layout.
- Nice work using the flexbox but the main target of the challenge is to use the grid layout which would have also helped you practice your skills using the grid which has boundless possibilities in terms of web layout.
Good effort so far.
Marked as helpful0@JayDeeZzzPosted over 2 years ago@coder-teamie First of all thanx for the comment buddy.
- Yeah! Totally forgot that. So I'm gonna fix it now :D
- Btw I used the grid layout instead of the flex boxes. Plz check the css code below.
.parent { display: grid; grid-template-rows: repeat(2,22rem); grid-template-columns: repeat(4,17rem); grid-column-gap: 2.5rem; grid-row-gap: 2.5rem; }
Cheerrzzzzzzzzzzzzz ๐ฅณ
0@tea-scriptsPosted over 2 years ago@JayDeeZzz That's good. Great job man ๐.
Marked as helpful0
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