Order Summary Page with HTML, and CSS

Community feedback

• 

  Rio Cantre•9,650

  @RioCantre

  Posted over 3 years ago

  Hello there! Nice work with this one. Viewing at your solution, I would suggest the following for you...

  • Remove the background-color: hsl(225, 100%, 94%); in the container and place is on the body rule set
  • Add a hover state for the #button1 and add background: hsla(245, 78%, 79%, 1);
  • In the #button2 add a background white
  • Wrap the whole content container with specific tag like main and attribution with footer tag for readability

  Overall you did well and Keep it up!

  03

  Marked as helpful
• 

  Web Wizard•5,690

  @rsrclab

  Posted over 3 years ago

  Hi, @webdevelopment4you ~

  Congratulate on your solution to the challenge on FM platform. I have studied your work carefully and learned a lot from it.

  Here are some of the tips I like to provide.

  • I suggest you to try transition on hoverable elements like cancel button or proceed button.
  • Please try BEM for naming element classes. It will help you a lot on bigger projects.

  If it can help you even a bit, it would be happy to me.

  Cheers ~

  01

  Marked as helpful