Responsive landing page using CSS Grid

Community feedback

• 

  Osaro Iyoha•1,080

  @osaarohPosted over 2 years ago

  Hello Josh,

  Welcome and good work in completing the challenge. Here are some suggestions:

  • If you want to center your card I recommend you to remove all the margin on "section", put <section> inside a <main class="card-main"> tag with the following attributes:

    display: flex;
    justify-content: center;
    align-items: center;
    min-height: 100vh;
  

  The first three is to center in the middle and the last one takes all the size of the height of the device that sees it and if the content is bigger the box is enlarged for that reason it is minimum of height.

  • You can use the "View Report" section for recommendations and implement them to make your code better

  Marked as helpful

  0

  ---

  

  Josh•40

  @jokingsPosted over 2 years ago

  @Master-Osaro Thanks

  0