Skip to content
  • Unlock Pro
  • Log in with GitHub
Profile
OverviewSolutions
7
Comments
1
P
Gabriela C
@gcardenasdev

All comments

  • nikos angelakis•60
    @Asaverle
    Submitted about 5 years ago

    Using CSS Grid

    1
    P
    Gabriela C•195
    @gcardenasdev
    Posted about 5 years ago

    Hi Niko! I really like how you used Grid to layout the cards!

    A few things I noticed:

    1)It was pointed out to me on my own project, the excessive use of H1. Per the MDN web docs the H1 tag should only be used once per page.

    2)Font-family is repeated every time you style your text. It would be cleaner to define it once like in the body selector. Same with the box-shadow and border radius for your cards.

    This is great work with only two weeks studying! Keep it up!

Frontend Mentor logo

Stay up to datewith new challenges, featured solutions, selected articles, and our latest news

Frontend Mentor

  • Unlock Pro
  • Contact us
  • FAQs
  • Become a partner

Explore

  • Learning paths
  • Challenges
  • Solutions
  • Articles

Community

  • Discord
  • Guidelines

For companies

  • Hire developers
  • Train developers
© Frontend Mentor 2019 - 2025
  • Terms
  • Cookie Policy
  • Privacy Policy
  • License

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub

Oops! 😬

You need to be logged in before you can do that.

Log in with GitHub