Order summary component

Community feedback

• 

  Adriano•38,040

  @AdrianoEscarabotePosted over 2 years ago

  Great job on this challenge! your report has some problems:

  Your card-container div got a little stretched and it wasn't very similar to the example, you could adjust its height so that it fits with the content, you could use a fixed height or some other responsive one that when the layout goes down it it is decreasing too, but congratulations the rest was very good.

  Hope it helps:)...don't forget to mark it as helpful 👍

  Marked as helpful

  1

  ---

  

  Mohamed Metwally•50

  @mmetwally0Posted over 2 years ago

  @AdrianoEscarabote Thanks a lot, will definitely adjust that.

  0
• 

  Luiza Uszacki•250

  @LuizaUszackiPosted over 2 years ago

  Hi, @mmetwally0.

  Instead of main .card-container .summary-box .payment a:hover try main .card-container .summary-box .payment > a:hover. It will change the color just of the payment's a direct child. So, you won't have your payment-btn's a changing its text color.

  You could also add a transition to make the change smoother.

  1

  ---

  

  Mohamed Metwally•50

  @mmetwally0Posted over 2 years ago

  @LuizaUszacki Thanks alot!

  1
• 

  Devmor•470

  @devmor-jPosted over 2 years ago

  Hey this is nice 🎉

  One small suggestion I can make is to remove height: 90vh from card-container class and instead add padding-bottom: 2rem.

  Everything else is working as expected, keep going and have fun 😃 🌹

  0