Order Summary using Flexbox

Community feedback

• 

  Daniel•170

  @Catalyst497Posted over 2 years ago

  Hi Marit,

  I just looked at your project solution and I must say, your solution looks very detailed. I also like the fact that you removed all the default stylings to every possible HTML element with the reset CSS. Yeah, those default stylings could be really troublesome sometimes :(

  Just to add, I think your body tag could use a little bit of margin to it. The way the content touches the top of the page might not look really good to the users. You could try adding a little bit of body{ margin: 1rem }

  I think you should add a common a class to elements if you want to give them a common style or better still I use this(I know it's a lazy developer thing): parent > * {property: style}. Now this is only a good fit if you want the direct children of an element to have a common style to them. It just works for me, you know.

  I hope you find this feedback helpful. Happy Coding.

  1

  ---

  

  Marit•400

  @MaritxxPosted over 2 years ago

  @Catalyst497 Hi! I see what you mean when it comes to the margin. On desktop this wasn't an issue but on mobile it would be, I had not really considered that. Thanks for the feedback! ^^

  1