Order Summary Card Component made by HTML and SCSS.

Community feedback

• 

  dewslyse•3,025

  @dewslysePosted about 3 years ago

  Hello @chetanhaobijam 👋! Congrats on your submission. Overall, nice job. Some things to note:

  • You could centre your component to the page with

  body {
      display: flex;
      align-items: center;
      justify-content: center;
      min-height: 100vh;
  }
  

  • Rather than use background-pattern with position: absolute , you could easily add the background image to the body in css

  body {
      background: url("./images/pattern-background-desktop.svg");
      background-repeat: no-repeat;
      background-size: contain;  
  }
  

  • The images are decorative and it's best for them to have alt=""
  • Since you've corrected the accessibility and html issues raised, why not generate a new report.

  All the best

  Marked as helpful

  0

  ---

  

  Haobijam Chetan Singh•195

  @chetanhaobijamPosted about 3 years ago

  @dewslyse Thanks for the feedback. I will see to it asap.

  0