Responsive Order Summary Component

Community feedback

• 

  Abdul Khaliq 🚀•72,560

  @0xabdulkhaliqPosted over 1 year ago

  Hello there 👋. Congratulations on successfully completing the challenge! 🎉

  • I have other recommendations regarding your code that I believe will be of great interest to you.

  ---

  BACKGROUND iMAGE 📸:

  • Looks like the background svg and color has not been set, let me explain How you can easily apply the background color with the svg they provided.

  ---

  • Add the following style rule to your css, and then experience the changes

  body {
      background: url(./images/pattern-background-desktop.svg), #E1E9Ff;
      background-repeat: no-repeat, no-repeat;
  }
  

  ---

  • Tip, Don't forget to generate a new screenshot after editing the css file

  ---

  .

  I hope you find this helpful 😄 Above all, the solution you submitted is great !

  Happy coding!

  Marked as helpful

  0

  ---

  

  P

  Orazbek•110

  @orazdaurenovPosted over 1 year ago

  @0xAbdulKhalid thank you for your feedback !! I've added the background, actually I forgot about the background.

  One more thing if I can ask you, as for the payment button, when the page is resized the button remains unchanged, even if I set it in REM, not pixels

  What should I add to match the design ?

  thanks

  0
• 

  P

  Genildo Cerqueira Souza•470

  @GenildocsPosted over 1 year ago

  Hello, sorry for the delay in responding. You can do it this way, put the content of the footer in a div, for example:

  <footer> <div class="">content</div> </footer> This way you can apply the display flex to the footer without affecting the elements contained in it.

  0
• 

  P

  Genildo Cerqueira Souza•470

  @GenildocsPosted over 1 year ago

  Hello, congratulations on completing the challenge!

  • I have a recommendation about the footer section, could you apply flex-box to it and center-align it, then give it a height so it doesn't get stuck to the main content. It also doesn't have enough width to grow and scale the content correctly.

  I hope I have helped.

  0

  ---

  

  P

  Orazbek•110

  @orazdaurenovPosted over 1 year ago

  @Genildocs thank you for your feedback, could you please explain it to me more precisely ? because i tried what you suggested, but don't see any changes

  0