Skip to content
Submitted about 4 years ago

CSS Grid, Flexbox, Sass

Klaudia•10
@klaudiaholyst
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi, I would be grateful for any advice! What to do better, what is wrong, what is good ;)

Thanks a lot!!!

Community feedback

  • Marlon Johnson•15
    @MvrsPosted about 4 years ago

    I like it!

    But, I would try adding more letter spacing to the card details, and adding more width to the cards.

    Overall, you're on the right track :)

    1
  • Klaudia•10
    @klaudiaholystPosted about 4 years ago

    @cakaragiannis @Mvrs thank you for your comments and advice! I appreciate it.

    I changed my solution following your advices. If you see there's still something to improve - let me know, if you want :) Thank you!

    0
  • Chris Karagiannis•65
    @cakaragiannisPosted about 4 years ago

    Well done on this. I agree with the previous comment. I would suggest adjusting font sizes to create a better visual hierarchy between the page and card headings. You could also soften the shadows.

    0

Please log in to post a comment

Log in with GitHub

Join 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