order-summary-component-main

Community feedback

• 

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  Replace the <figure> with the main tag to fix the accessibility issues. for more on semantic html visit https://web.dev/learn/html/semantic-html/

  Reduce the width of the figure to 25vw / 20vw. To center the figure on the page, add min-height:100vh; display: flex; align-items: center: justify-content: center; to the body. instead giving figure a margin value to center it.

  You forgot to add a box-shadow to proceed payment button

  Hope am helpful. Happy Coding..

  0

  ---

  

  ِAbdullah•730

  @9dorghamPosted almost 2 years ago

  @Hassiai Thank you for your attention to my code it's really helpful. I know the way that it is use to center the element using flex-box, but I don't like using it too much in my code

  0

  

  Hassia Issah•50,650

  @HassiaiPosted almost 2 years ago

  @9dorgham , Use min-height:100vh; display: grid place-items: center instead of flexbox

  if my comment is helpful , mark it as helpful.

  Marked as helpful

  0