it's an intro component with a sign-up form, using HTML, CSS, JS

Community feedback

• 

  Alberto Ledesma•370

  @ledesmxPosted about 1 year ago

  Hi Yasmin 👋

  Great job on your solution!

  Here are some recommendations for you:

  • I found that there are two background images when the screen's width is shorter than 650px. Because the background images are rendered in two different elements (body and .container). I suggest removing one of them:

  @media (max-width: 650px) {
    .container {
      background-image: url(images/bg-intro-mobile.png); /*Remove this*/
    }
  }
  

  • Whether you want to render one background or the other. You can add the media query for the body tag. Like this:

  body {
    background-image: url(images/bg-intro-desktop.png);
  }
  @media (max-width: 650px) { /*Add this media query*/
    body {
      background-image: url(images/bg-intro-mobile.png);
    }
  }
  

  I hope this helps a little.

  Well done for the rest.

  0