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

@nicolasgonzalez98

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


I need help with the image of the girl and the double bg image. Thank you so much and sorry for my english.

Community feedback

Travolgi 🍕 31,500

@denielden

Posted

Hi Nicolás, I took some time to look at your solution and you did a great job!

Also I have some tips for improving your code:

  • add descriptive text in the alt attribute of the image
  • for fix the size of top image adding background-size: contain to main tag
  • remove height from card class because it is superfluous
  • use button element and not div for "Proceed to Payment"
  • try to add hover effect and a little transition on the button
  • instead of using px try to use relative units of measurement -> read here

Overall you did well :)

Hope this help and happy coding!

Marked as helpful

1

@nicolasgonzalez98

Posted

@denielden thank you, i will take all your advices :)

1
Abhik 4,840

@abhik-b

Posted

👋 Hello Nicolas , Your solution looks nice 😃

  • I think if you use img tags in html instead of background-image in CSS for the girl image then it will look even nicer & solve the problem.
  • for double bg image ( I assume the background image you are referring to) you can use background-size:contain inside main to make it look nicer
  • I really think you should use media-query for responsiveness & stay away for viewport units vw for width & height of simple elements like card

Other than that overall it looks nice so Please keep solving challenges & happy coding 🥳👍

Marked as helpful

1

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