Skip to content
Submitted about 3 years ago

3-Column-Preview-Cards

Keizzmann•110
@Keizzmannbanjo
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...3-column preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Hello community,

So i decided to test my basic skills by creating this card design.

I would love the community to go through my codes and give suggestions on making things better

Thanks folks xoxo

Community feedback

  • Kylon Tyner•30
    @tynerk92Posted about 3 years ago

    It looks pretty good!

    My two cents would be:

    1. Have some more discerning icons, they all look pretty similar
    2. The buttons look much better in the design than the implementation. Some more padding should help out with that :)

    Great work!

    1
  • Dušan Lukić•1,660
    @dusanlukic404Posted about 3 years ago

    Hi Keizzmann, good work but I have a few suggestions:

    • Add more padding top and bottom on your buttons
    • Add transition property to button on :hover
    • You didn't use regular font-family for <h1>
    • Reduce the line height of <p> elements
    0
  • darryncodes•6,390
    @darryncodesPosted about 3 years ago

    Hi Keizzmann,

    Nice work, if you swap <div class="container container_flex"> for <main class="container container_flex"> you'll clear your accessibility report. Using descriptive mark up is really important.

    All the best!

    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