Not great Not terrible 3.6

Community feedback

• 

  Rohit Kumar•560

  @rohitKumar38344Posted over 2 years ago

  1. Remove the test div it's unnecessary placed in the solution.
  2. Remove the display: flex property from main it's not necessary
  3. Use vw instead of px to apply background image with respect to viewport

  body {
      background: url("images/bg-pattern-top.svg"),url("images/bg-pattern-bottom.svg");
      *** background-position: top -66vw left -46vw, bottom -76vw right -38vw; ***
      background-repeat: no-repeat;
      background-size: cover;
      background-color: var(--cyan);
      background-blend-mode: overlay;
      display: flex;
      height: 100vh;
      justify-content: center;
      align-items: center;
      font-family: 'Kumbh Sans', sans-serif;
  }
  

  Marked as helpful

  0

  ---

  

  Danteie•300

  @DanteiePosted over 2 years ago

  @rohitKumar38344

  Thank you, is this good now ?

  0

  

  Rohit Kumar•560

  @rohitKumar38344Posted over 2 years ago

  @Danteie yes it's good now

  0