order summary component solution

Community feedback

• 

  Bernardus•1,095

  @BernardusPHPosted over 2 years ago

  hey GUILLERMO MULVIHILL. I think you might like this approach bc I see that your background img does not reach the end horizontally and its much easier centering. Hope this helps

  body {
      background-color: var(--Pale-blue);
      background-image: url(images/pattern-background-desktop.svg);
      background-repeat: no-repeat;
      background-size:contain;
      min-height: 100vh;
      display: flex;
      justify-content: center;
      align-items: center;
  }
  /* main removed */
  
  

  0

  ---

  

  Guillermo Mulvihill•190

  @guilleoemPosted over 2 years ago

  @BernardusPH But, is not a good practice to have a main landmark?

  0

  

  Bernardus•1,095

  @BernardusPHPosted over 2 years ago

  @guilleoem Oh sorry I was not specific I meant I did not add any css to the main, I still kept the main element in the html. Sorry again

  1

  

  Guillermo Mulvihill•190

  @guilleoemPosted over 2 years ago

  @BernardusPH Anyway, thanks for remind me the background-size property.

  1