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

All comments

  • Stone Leiderson• 45

    @stone-leiderson

    Posted

    Great job on the layout! I noticed the background color is different in the header section. If you fix that to match the rest of the body's background it would look great!

    0
  • Stone Leiderson• 45

    @stone-leiderson

    Posted

    Good job on the solution!

    Some things I noticed in your code was repeating items or unnecessary classes like .top-cyan-line, etc. Instead, use the border-top property on your .card-1 classes.

    Also, for the img class, you can group them into one class instead of writing the same properties for each image.

    Definitely try grouping and condensing as much of your properties as possible to avoid repeating your code.

    Other than those minor custodial changes, you matched the design very closely! Nice work.

    1
  • Stone Leiderson• 45

    @stone-leiderson

    Posted

    For the cards, I would try using text-align: left; to get the text in the boxes to align left.

    Also, try condensing or grouping similar styles together. You repeat yourself a few times in your css. Other than that, good job on completing the challenge!

    0