Mobile first approach with Css Grid for Layout.

Community feedback

• 

  Naveen Gumaste•10,480

  @NaveenGumastePosted over 2 years ago

  Hello Ernest Push Gbaboh ! Congo 👏 on completing this challenge

  Let's look at some of your issues, shall we:

  • The background color is given in style-guide.md

  • Why us section color is too much light and i know it was not given in style-guide so use color-zilla chrome extension. and you will get the color

  • This is color i used for my solution --light-cyan: hsl(179, 45%, 52%); check my solution here if needed

  • To center the card vertically

  max-height: 100vh;
  display: flex;
  justify-content: center
  align-item: center;
  

  happy Coding😀

  Marked as helpful

  0
• 

  Shashree Samuel•9,260

  @shashreesamuelPosted over 2 years ago

  Hey gbabohernest, good job on completing this challenge. Keep up the good work.

  Your design looks good however I think your card needs some space on the margin-top. This can be achieved using the margin-top css property.

  In terms of your accessibility issues simply wrap all your content between <main> tags.

  I hope this helps

  Cheers, Happy coding 👍

  Marked as helpful

  0