Skip to content
Submitted over 3 years ago

My solution to grid cards

Web Wizard•5,690
@rsrclab
A solution to the Testimonials grid section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Any feedback is welcome here!

Code

Please log in to post a comment

Log in

Community feedback

  • Travolgi 🍕•31,300
    @denielden

    Hi Tymur, great job! The hover effect is fanstastic :)

    You seem to have forgotten about the background-color in the body.

    To center the elements on the page try using Flexbox, it can help you better: give the flexbox and height properties to the body and remove margin from the .grid-cards class.

    Read the guide -> Flexbox

    Note: Flexbox aligns to the size of the parent container.

    You can use the vh measurement for the height... Viewport Height handles the sizing of an element in relation to the height of the browser window.

    Hope this help ;)

    Marked as helpful
  • LiBee•370
    @Li-Bee

    Looks very good - well done! I dont think the background-color on the <body> has been applied.

    Also have accessbility issues need to solve. I think if you add <main> to the <body> it should solve the issues.

    Marked as helpful
  • Kamasah-Dickson•5,570
    @Kamasah-Dickson

    I think a little bit of margin to the card bottom is needed Good job:)

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