Mobile first order summary component using Sass

Community feedback

• Account deleted

  Hi there 👋

  Congratulate on finishing your project 🎉. You did a great job 💡.

  I give some suggestions to help you take your project design to the next level 📈😉

  • One thing left there is the color of the box-shadow in the cards should not be black, change it to dark blue or as in the button
  • Also add cursor: pointer to the button

  Happy coding ☕

  Maqsud

  Marked as helpful

  0
• 

  Rio Cantre•9,690

  @RioCantrePosted almost 3 years ago

  Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…

  • Remove unnecessary code to keep it clean
  • Add the background image of the design. Include it inside the body rule set
  • Add cursor: pointer for the main button

  Above all, Well done! Keep it up and Hope this is helpful!

  Marked as helpful

  0
• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

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

  Also I have some tips for improving your code:

  • try to add a little transition on the element with hover effect
  • For add the top image in the background just put more specific background properties to the body:

  background: url("../img/pattern-background-desktop.svg") no-repeat top center;
  background-size: contain;
  background-color: #e0e8ff;
  

  Overall you did well :)

  Hope this help and happy coding!

  Marked as helpful

  0
• 

  Ruan Esterhuyse•170

  @RuanEsterhuysePosted almost 3 years ago

  Thanks for the feedback! I appreciate it.

  0