Mobile first order summary component using HTML and CSS

Community feedback

• 

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  Hi yjkim!

  You have done quite well, I like it, it has no accessibility errors. I would only say that you should correct the background.

  Put these two attributes to your body:

  background-size: contain;
  background-color: hsl(225, 100%, 94%);
  

  You could also add a nice little shadow to your card as follows:

  box-shadow: 0 1.5em 1em hsl(227, 72%, 87%);

  If you want to give it a more professional touch add transition to all the links and buttons adding this attribute like transition: all 500ms, this way you add a transition so that it is not so abrupt the changes of style when you pass the cursor or you leave pressed.

  Marked as helpful

  1

  ---

  

  yjkim0109•60

  @yjkim0109Posted over 2 years ago

  @Sdann26 Wowow! After following your suggestion, It looks much better than before :)! I really appreciate your help.

  1

  

  Danilo Blas•6,300

  @Sdann26Posted over 2 years ago

  @yjkim0109 Hehehe you are welcome. If you can generate a new Screen Shot, so that the preview of your project looks according to the changes you have made. If you are not a premiun frontend mentor user you get 5 free per month :D.

  Marked as helpful

  1

  

  yjkim0109•60

  @yjkim0109Posted over 2 years ago

  @Sdann26 Oh Thank you so much!!

  1