Order Summary Card with html and css

Community feedback

• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  I suggest you center the component on the page by giving the parent element <main> the following properties:

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

  I think you forget to set the background... set it up, background-image: url(../images/pattern-background-desktop.svg);

  Marked as helpful

  1

  ---

  

  Rocky Gurung•50

  @grgrockyPosted over 2 years ago

  @Bayoumi-dev Thanks mate for your kind suggestions. I did it like you said. It worked. But even after 'Generating Screenshot' the issues reports are still the same. I don't know why, may be I'm not a pro member. However, Thanks for your time.

  Cheers

  0
• 

  Ahmed Bayoumi•6,740

  @Bayoumi-devPosted over 2 years ago

  Hey Rocky Gurung, Congratulations on completing this.

  • It looks great! but you have some issues you need to fix.
  • Heading levels should only increase by one, Change h3 to h2
  • Document should have one main landmark, Contain the component in `<main>.

  <main>
     <section id="container">
        //...
     </section>
  </main>
  

  Keep coding👍

  Marked as helpful

  0

  ---

  

  Rocky Gurung•50

  @grgrockyPosted over 2 years ago

  @Bayoumi-dev thanks Ahmed for pointing out the issues and your kind suggestions.

  Cheers

  0