Order summary component

Community feedback

• 

  David•7,960

  @DavidMorgadePosted over 2 years ago

  Hello Abdullahm congrats on finishing the challenge!

  I will try to answer your questions.

  1- I think that the path that you took using double flex-box is the best way to do it, you could try to adjust it with some margins and paddings but is better to just use flex to get the desired result.

  2- Try using some CSS variables for your colors, so any time you have to use a color you don't need to use the full hsl syntax, and also, for future apps, you can change the full theme of your webpage just changing 3 or 4 values, instead of all the places where you put colors, if you are not familiar with CSS variables, check-this-out

  3- I would have change the first div that has the whole component inside into a main tag, then the image separated at the bottom with some semantically related section, but this is just a little project, you could also use divs for the inside parts of the card. Would recommend you to use at least one h1 tag, probably 'Order Summary' fits better as an h1.

  My last advice would be to add some cursor: pointer; to be clear to the users that this buttons are clickables!

  Hope my feedback help you! great work!

  0

  ---

  

  abdullahmd2•90

  @abdullahmd2Posted over 2 years ago

  @DavidMorgade Thank you so much for answering my questions.

  0