Skip to content
OverviewSolutions
2
Comments
4

Kenbak

@Kenbak40 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • Kenbak•40
    @Kenbak

    Hello friend! Congrats on your first challenge!

    You might want to change the background color to:

    body {
  background-color:hsl(212, 45%, 89%)
}

    And the paragraph color to a lightgray!

    You can also use this code to center your elements:

    body {
    min-height: 100vh;
    display: grid;
    place-content: center;
}

    Hope it helped, and will be looking forward to your next challenges!

  • Kenbak•40
    @Kenbak

    Congrats on your first challenge ser. It's very impressive. The card is the exact same size.

    Instead on using margin top and bottom on the texts elements, I would put them all in a div, using flex and flex-column. Then add a justify-content-around and it should do the trick and be more symmetrical!

    Also you can reduce a bit the line height on the H1 to make it perfect ;)

    Marked as helpful
  • Kenbak•40
    @Kenbak

    Hello Saadi,

    You can also use this code to center the elements:

    body {
    min-height: 100vh;
    display: grid;
    place-content: center;
}

    The card also looks a bit small, you might want to make it bigger! It's a great idea to have added a bit of shadow, but you might want to make it a tiny bit less visible. Congrats on your first challenge!

  • Kenbak•40
    @Kenbak

    Congrats on your first challenge friend!

    I would put this code in the body selector so the component is centered.

        min-height: 100vh;
    display: grid;
    place-content: center;
}

    Also you might want to use a separate css stylesheet so your code is more readable. It doesn't matter for small projects like this one, but it will become hard to follow through on more developed apps.