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

Mobile first design using flexbox

ccbdp 315

@ccbdp

Desktop design screenshot for the Order summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Any type of feedback will be highly appreciated!

Community feedback

dewslyse 3,025

@dewslyse

Posted

Hello 👋! Nice job on your submission.

Some minor things to note:

  • Your could center the card to the page by adding the following the body
body {
  display: flex;
  justify-content: center;
  align-items: center;
  min-height: 100vh;
  flex-direction: column;
}

Or you could just change the html in your css to body

  • You could use more semantic html elements. At least wrapping your main content within the main tag would help resolve some of the accessibility issues raised in the report.
  • Wouldn't it be more appropriate to use an <a> instead of a <button> for Proceed to Payment ? Proceed somehow implies navigation

Marked as helpful

0

ccbdp 315

@ccbdp

Posted

@dewslyse Thank you for your feedback! I would definitely apply the concepts you said when I edit this project and the projects I make moving forward!

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