Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Testimonial Grid

Nikki 80

@N11KK11

Desktop design screenshot for the Testimonials grid section coding challenge

This is a solution for...

  • HTML
  • CSS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


I forgot to place background apostrophe img on Daniel Clifford's testimonal. I will edit soon. This was good practice for learning CSS grid. Any advice is much appreciated.

Community feedback

@catherineisonline

Posted

Looks cool! I would also add some margin or padding at the end of the container so there is some space. And add lang="en" to the HTML tag to get rid of the reported error 👍

Marked as helpful

1
Adriano 34,090

@AdrianoEscarabote

Posted

Hello Nikki, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:

To align some content in the center of the screen, always prefer to use display: flex; it will make the layout more responsive!

Example:

body {
    margin: 0;
    padding: 0;
    display: flex;
    align-items: center;
    flex-direction: column;
    justify-content: center;
    min-height: 100vh;
}

<html> element must have a lang attribute, to inform the language of the page content!

to solve this use the attribute in the html tag;

<html lang="en">

The remainder is excellent.

I hope it's useful. 👍

Marked as helpful

1

Please log in to post a comment

Log in with GitHub
Discord logo

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