Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

order-summary-component-main

ِAbdullah 730

@9dorgham

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Hassia Issah 50,670

@Hassiai

Posted

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

@9dorgham

Posted

@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,670

@Hassiai

Posted

@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

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord