Order summary component

Community feedback

• 

  Adriano•42,870

  @AdrianoEscarabote

  Posted almost 3 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 👍

  11

  Marked as helpful
• 

  Luiza Uszacki•250

  @LuizaUszacki

  Posted almost 3 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.

  11
• 

  Devmor•470

  @devmor-j

  Posted almost 3 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 😃 🌹

  00