Landing page

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace the section tag with the main tag to fix the accessibility issues. there is no ne for <a> within the <button> remove it to fix the error issues. in the css give the button a cursor value of pointer.

  for the right and left-sides of the image give the nav, the main and the footer a max-width of 1440px, a width of 80% and margin: 0 auto.

  nav, footer, main{
  max-wisth:1440px;
  width: 80%;
  margin: 0 auto;
  }
  

  change the value of background size in the mobile design to contain and in the desktop design to cover.

  in the desktop design give .landing_img a width of 55% and .landing_text a width of 40%. and no need for margin-right value in .landing-text.

  Hope am helpful.

  Well done for completing this challenge. HAPPY CODING

  Marked as helpful

  1
• 

  Redha•90

  @med-redhaPosted almost 2 years ago

  i think if u add &.logos class name : hover {box-shadow } and a transition to the logos class name would do the trick .

  1