Summary Order

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Hey there,

  congrats on completing the challenge successfully! Your submission looks pretty nice overall.

  It looks like your background color is missing on the desktop view. You can just add this: background-color: #e0e8ff; to the body to solve and match the design. But if it is your preference to keep it white, you can also try adding a box-shadow e.g. box-shadow: rgb(149 157 165 / 20%) 0px 8px 24px; to the .card for an interesting look. Here is a great resource of some cool box-shadows, i also use a lot: https://getcssscan.com/css-box-shadow-examples

  Hope you find this feedback helpful!

  Marked as helpful

  1

  ---

  

  Barry Thatcher•70

  @Bazza-08Posted over 1 year ago

  @visualdenniss Thank you!

  0
• 

  P

  markus•2,250

  @markuslewinPosted over 1 year ago

  Try using padding, instead of margin, for space between a container and its content!

  0
• 

  Atif Iqbal•3,200

  @atif-devPosted over 1 year ago

  Hi Barry, congrats🎉 on completing the challenge.

  • Learn The var() Function from here to tackle DRY in your css.
  • For background modify body selector code as:

  body{
      margin: 0;
      padding: 0;
  
      text-align: center;
      background: url("../images/pattern-background-desktop.svg") no-repeat top center;
      background-color: hsl(225, 100%, 94%);
  
      display: grid;
      min-height: 100vh;
      place-content: center;
  }
  

  After applying above block of code also remove following code:

  @media(max-width: 700px) {
      
      body {
          background:hsl(225, 100%, 94%);
      }
  }
  

  Hope you will find this Feedback Helpful.

  0