Four card feature section using Css Grid

Community feedback

• 

  P

  Julie•130

  @salentipyPosted 6 months ago

  Hello! I'm doing one of Frontend Mentors learning paths, and part of that is giving feedback on someone else's work, and they chose yours for me to give feedback on. Having said that, I'm still pretty much a newbie when it comes to responsive layouts, so I don't really have too much I can say about your work. You succeeded in creating the layouts as required for both layouts, so most of what I would offer would just be my opinion on differing ways to achieve things. So, I really only have two things to suggest. First one, your header in your mobile layout needs to be centered. Second, you could achieve the colored top on each card a simpler way by creating a color class and using "border-top." But, like I said - just my opinion! Overall, this looks great and keep up the good work!

  0
• 

  Abdulazeez-20•60

  @Abdulazeez-20Posted 6 months ago

  Your project looks great!

  I have a suggestion about your code that might interest you: *change the first heading opacity so it would look similar to the design *add a margin or padding after the heading *the colors of green, cyan cards should be cyan, blue *the shadow looks really sharp try to add some blur to it

  0