All comments

• 

  gabeporto•20

  @gabeportoSubmitted almost 3 years ago

  

  Order Summary

  1

  30
  

  Attila•290

  @unexunderscorePosted almost 3 years ago

  Hi there! Great job! Keep going!

  Some remarks: Background image missing (wave) Annual plan text should be also bluish colored. Around the card layout should have some shadow.

  Have a nice day!

  Marked as helpful

  0
• 

  Andy Ouyang•30

  @ahouyangSubmitted almost 3 years ago

  Any general tips on code styling/best practices are always appreciated. I also used flexbox basically any time I needed to move a component around, wasn't sure if that's always ideal or not. Thanks!

  

  Order summary challenge

  2

  40
  

  Attila•290

  @unexunderscorePosted almost 3 years ago

  Hi there! Great work, just keep going!

  Some remarks:

  1. Title (Order Summary) and the Annual Plan text should be different color (bluish).
  2. If you hover over your mouse on the change link, you should not see underline. (text-decoration: inherit; )
  3. Where the music icon and annual plan text etc. There should be a very light gray box around, check the design picture, you can see slightly.
  4. Box shadow under the card layout and the button. (Sample: box-shadow: 0 8px 16px 0 rgba(0,0,0,0.2);)

  Keep coding!

  Marked as helpful

  0
• 

  Samsoor•120

  @Samsoor-nanSubmitted almost 3 years ago

  Dear All,

  I tried my best to complete this solution, however the image at the top seems to get reduced from right when screen size reduced. while I have set the image width to 100%. kindly guide.

  Thanks,

  

  HTML,CSS

  2

  30
  

  Attila•290

  @unexunderscorePosted almost 3 years ago

  Hi there! Nice work, just keep going! I’m a beginner, but I work every day to get better, so keep it up!

  My comments: Try to set the background-size: 100% I think this should solve the background size problem.

  Also if you hover over the change link, you should remove the underline. Use this if you hover -> text-decoration: inherit;

  Have a nice day!

  0