Responsive Landing Page Using Css Grid

Community feedback

• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  👾Hello Joy, congratulations for your solution!

  I saw that you've used the shorthand to apply the background with the image top and bottom, the way you did is kinda tricky to manage the images because there are many properties to insert, my advice for you is manage each background property separately.

  I did a fix for your background and you can see the code below:

  Body {
      background-image: url(../images/bg-pattern-top-desktop.svg), url(../images/bg-pattern-bottom-desktop.svg);
      background-position: left -185px top -236px, right 10px bottom -300px;
      background-repeat: no-repeat, no-repeat;
      background-attachment: fixed, fixed;
      background-size: contain, contain;
  }
  
  

  Hope it helps and keep it up!

  1