Responsive clipboard-landing-page-master

Community feedback

• 

  バレンタイン 😈•64,170

  @VCaramesPosted about 2 years ago

  Congrats @ky302 on completing this challenge!

  Looks good!

  In your header in of using a div for the <div class="header-sec"> you can instead use a <section> element.

  You also want to use an <h1> for the "A history of everything you copy" instead of an <h2>. When building landing page, like this one, you always start with <h1> in your hero section.

  For the background image in your header, add the following to your <header> element:

  header {
      background-position: center;
      background-size: cover;
  }
  

  Happy Coding!

  Marked as helpful

  0

  ---

  

  Ahmad Subekhi•150

  @ky302Posted about 2 years ago

  @vcarames thanks brother for the very helpful advice, I will try it soon.

  0