Skip to content
OverviewSolutions
24
Comments
30
P

AjeaS

@AjeaSmithYpsilanti, Michigan755 points

Hobbyist dev learning as I go.

I’m currently learning...

To become a better frontend developer by building and practicing as much as I can :)

Latest solutions

View more solutions

Latest comments

  • P
    AjeaS•755
    @AjeaSmith

    Awesome work on the layout!

    Some areas you can amend:

    • The correct background color for the patrick adam card can be found in the style-guide.md file of the project.
    • The margin from the h2 in your .grid__user class is causing extra spacing between text, I would suggest removing that margin.

    Great work, happy coding.

  • Lawrence Webber•160
    @lwebbz
    What are you most proud of, and what would you do differently next time?

    First time using css grid

    Four Card Feature

    1
    P
    AjeaS•755
    @AjeaSmith

    Awesome!

    Minor Improvement:

    • Small suggestion to add some space between the header text and the cards.
  • P
    AjeaS•755
    @AjeaSmith

    Great work!

  • P
    AjeaS•755
    @AjeaSmith

    Nicely done, for the responsive portion I would suggest looking into Flexbox or Grid along with using media queries (which involves changing the layout on different screen sizes).

  • P
    AjeaS•755
    @AjeaSmith

    Really nice job on this! :D

  • P
    AjeaS•755
    @AjeaSmith

    Thanks! :) and good catch I’ll check that out. Yea it took me quite some time to get the background image to work.

View more comments