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
Request path contains unescaped characters
Request path contains unescaped characters
Request path contains unescaped characters

Submitted

Order summary component

@jilliannoreen

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 would love any feedback or suggestions to improve my code. Thank you in advance!

Community feedback

Account Deleted

Hello there! 👋

Congratulations on finishing your challenge! 🎉

I have some feedback on this solution:

  • add cursor pointer to button so that the user knows its clickable

  • add tranistion:1s; to the button so that when you hover on the button the background changes smoothly

i hope this is helpful and goodluck!

Marked as helpful

0

@jilliannoreen

Posted

@Old1337 Hi! Thank you so much for the feedback. I've made the changes you suggested, and I think it looks better now. Thanks again for the help!

1
Divine Obeten 2,415

@Deevyn9

Posted

Hi Jillian,

The build itself is perfect, it has no issues. However, the attribution can be made to appear below the div which would make it look better

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