A responsive landing page with Grid

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Hey @hormakery, some suggestions to improve you code:

  • This entire component should have been built using CSS Grid as the challenge name implies. If you are not comfortable using CSS Grid or do not know how use it, here is a link that is all about how to use CSS Grid:

  https://css-tricks.com/snippets/css/complete-guide-grid/

  • To center you content to your page, add the following to your Body Element:

  body {
      min-height: 100vh;
      display: grid;
      place-content: center;
  }
  

  • There are three headings in this component; “Join our community”, “Monthly Subscription”, and “Why us”. They need be wrapped in a header element.

  • The button was created with the incorrect element. When users click on the button they should directed to a different part of your site; that Anchor Element will allow this to happen.

  • Your button needs to have box-shadow, cursor: pointer and a :hover.

  Happy Coding! 👻🎃

  0

  ---

  

  Bee Horma•170

  @hormakeryPosted about 2 years ago

  @vcarames thank you I didn't take note of that , I will make changes.....thanks for the feedback

  0