Skip to content
Submitted over 4 years ago

Using CSS Grid

nikos angelakis•60
@Asaverle
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Four card feature section
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

This is my first attempt to build a website after two weeks of reading about html and css. Please do rate my work and let me know if code could be cleaner and if there are any mistakes that I should avoid in the future.

Community feedback

  • Gabriela C•195
    @gcardenasdevPosted over 4 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!

    2
    nikos angelakis•60
    @AsaverlePosted over 4 years ago

    @gcardenasdev really appreciate your feedback. These are very useful. Will definitely look upon them right away and make necessary corrections. Many thanks again!

    0

Please log in to post a comment

Log in with GitHub

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