Order Summary Component solution

Community feedback

• 

  Sherif Lawal Itunu•230

  @lawal-sherif-itunuPosted about 2 years ago

  This is a nice work, keep it up. But, it is cool to note that it isn't a nice way of practice to use px in your work rather than rem or em. "Rem and pixel (px) are two units of measurement frequently used in CSS and HTML to define font size, margin or image size for example. However, the two units have a big difference in terms of web accessibility, because a pixel is not responsive and that's why it's better to use REM." Also, you shoudn't have used a fixed width on the card. you could have used stuff like (width: min(90%, 22rem);). Also, you forgot to give the button with the class of "cancel" a padding. do that

  0

  ---

  

  Agnar•220

  @agnar-nomadPosted about 2 years ago

  @lawal2000 Hi

  thank you for your feedback. I will keep your notes in mind with my next projects. Cheers

  0