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 Card

JJ Fuentes 140

@jjfuentes13

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


What I found difficult in this challenge was to use the drop-shadow / box-shadow property on the Proceed to Payment button. I tried both properties, but neither worked, and developer tools showed me an invalid property value triangle.

I also found this challenge easier to use flexbox instead of grid, as it was easier for me to size the elements. What tips or properties would have made grid work for this challenge?

Community feedback

@0xabdulkhaliq

Posted

Hi there 👋:

  • After inspected your code, i want to share my suggestions that will be improve your solution

SUGGESTIONS:

  • The button .proceedbutton wants some shadow
  • So kindly change the try out the following css which drops shadow down with less opacity
.proceedbutton {
    box-shadow: 0 10px 30px hsla(245, 75%, 52%, 0.2);
}
  • And finally, consider to remove the .attribution (footer) for better viewing experience

MESSAGE:

  • If my answer helps you then providing an upvote will be very gratefull
  • And don't forget to mark it as helpfull
  • I hope you learned a lot of stuffs during this project, Congrats JJ

GREETINGS:

  • Happy coding 🙌
  • Peace be upon you with god's mercy & blessings Mayordey..✨

Marked as helpful

1
Hassia Issah 50,670

@Hassiai

Posted

To center. container on the page, add min-height:100vh; and align-items: center; to the body

Replace the width with a max-width value for a responsive content , there is no need for a height value in .container.

Change the background-size of the body to contain , the body of the mobile design also requires a background-image. there is no need to give .container a width and height value in the media query.

Use rem or em as unit for the padding, margin and width for more on css units watch this https://youtu.be/N5wpD9Ov_To

Hope am helpful HAPPY CODING

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