Skip to content
Submitted almost 3 years ago

3-column-card-Solution

Stanhope Jite•90
@JyteCeo
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

we keep getting our hands dirty with code .. how far have I gone guys ..?

Community feedback

  • Rio Cantre•9,690
    @RioCantrePosted almost 3 years ago

    Hello there! Good job with this one. Looking at your solution, I would suggest the following for you...

    • Add margin-top: 6rem; in .main-container rule set to align the content in the center and adjust the border radius into 0.7em
    • Add line-height: 1.6; in the page-content rule set. Alternative is to wrap it with p instead of div
    • Add border radius in the media queries
    • Wrap the main-container with specific tag like main and attribution with footer tag for readability

    Hope this helps and Keep up the good work!

    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