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 only css html

gurpreet 280

@justcoder42022

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


always welcome feedback for improvement.

Community feedback

Mo 840

@MohamedAridah

Posted

Hello @justcoder42022, you did good job for this challenge.

However i have some notes:

  • change height: 100vh of the body to be min-height: 100vh. this will allow content to take more height of the body if it needs to.

  • use transition property for smoother hover effect.

  • use overflow: hidden for .wrapper. by doing this you don't need to set border-radius for the image.

  • there is extra space on the top and bottom of the main element. try to know it's source and remove it.

  • you can see My solution for this challenge it may be helpful for you..!

I hope this wasn't too long for you, hoping also it was useful😍.

Goodbye and have a nice day.

Keep coding🚀

Marked as helpful

1

gurpreet 280

@justcoder42022

Posted

@MohamedAridah thanks dear for your time

0
Mo 840

@MohamedAridah

Posted

@justcoder42022 you are welcome🌹👍

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