Skip to content
OverviewSolutions
20
Comments
3
P

ania

@ania221BPoland330 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

View more solutions

Latest comments

  • P
    ania•330
    @ania221B

    Hello Candido 👋

    Congratulation on completing the challenge! It looks really nice 🙂

    I've noticed that you often set height on elements. It's a good idea to leave that out and instead let the height be decided by the contents of the element and things like padding. So to make an element take up more vertical space, you could give it bigger padding, add margin to elements inside, increase line-height of the text and so on. I hope this helps.

    I apologize for commenting in English, but I don't know Spanish.

    Happy coding 🙂

    Marked as helpful
  • P
    ania•330
    @ania221B

    Hello Modade!

    To answer your question: yes, you can place the cards with CSS Grid. One way to do so would be defining rows and columns for the cards:

    .grid_card_one {
grid-row: 3 / span 2;
grid-column: 1;
}

.grid_card_two {
grid-row: 2 / span 2;
grid-column: 2;
}

.grid_card_three {
grid-row: 4 / span 2;
grid-column: 2;
}

.grid_card_four {
grid-row: 3 / span 2;
grid-column: 3;
}

    Here's a CSS Grid cheatsheet

    I hope this helps 😊

  • P
    ania•330
    @ania221B

    Hello @GreatNation111👋

    You've mentioned that the spacing between the paragraph and header is bigger than you wanted. This might be due to line-height set on <h1>. I think currently it is the same as for the rest of text content. It is worth decreasing it, as smaller line-height makes big text look better. Try changing it to 1.2 or similar value and see if it helps. Happy coding 😊

    Marked as helpful