Order summary component using html & css

Community feedback

• 

  Travolgi 🍕•31,400

  @denieldenPosted almost 3 years ago

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

  Also for fix 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

  ---

  

  kasemkl•40

  @kasemklPosted almost 3 years ago

  @denielden thanks a lot :)

  1
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted almost 3 years ago

  If you want to reduce accessibility to change this code 😊

  <div class="attribution"> to <footer> ( Used footer tag );

  Marked as helpful

  0

  ---

  

  kasemkl•40

  @kasemklPosted almost 3 years ago

  @GitHub-dev12345 thanks a lot .

  0
• 

  Dev Rathore•2,600

  @GitHub-dev12345Posted almost 3 years ago

  Congratulation to complete the challenge ❤️👍 My small suggestion : 1.In Card design Used This CSS Code:

  transform : scale(0.8); this property decrease the size of card. 😉

  large size for increase the number of scale & small size for decrease the number of scale

  I hope you find this helpful

  Marked as helpful

  0

  ---

  

  kasemkl•40

  @kasemklPosted almost 3 years ago

  Hi @GitHub-dev12345 , thanks a lot for your note ❤️👍

  0
• Account deleted

  Hello there! 👋

  Congratulations on finishing your challenge! 🎉

  I have some feedback on this solution:

  • Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info

  i hope this is helpful and goodluck!

  Marked as helpful

  0

  ---

  

  kasemkl•40

  @kasemklPosted almost 3 years ago

  @Old1337 thank you so mush

  1