All comments

• 

  Okan Öztürk• 120

  @joxinyks

  Submitted 7 months ago

  

  Order Summary Component by joxinyks

  1

  4

  0

  

  Miguel Ángel Ramírez Ruiz• 50

  @Miguelo0098

  Posted 7 months ago

  Good Job! It looks good overall. Good things to point out:

  • Use of CSS variables
  • Nice semantic HTML and class names
  • Use of media queries

  Some details can be improved:

  • Font is not displayed properly on devices without 'Red Hat Display' (including it inside the HTML head will make it work).
  • Hover states and button styles don't match the design, but that is just me being picky.
  • Card description needs a little more line-height (according to design).

  Marked as helpful

  0
• 

  FanisFFF• 230

  @FanisFFF

  Submitted 7 months ago

  

  Responsive Order summary component using Css

  1

  4

  0

  

  Miguel Ángel Ramírez Ruiz• 50

  @Miguelo0098

  Posted 7 months ago

  The solution looks good overall. Proportions are okay and I think the HTML is semantic enough. Good Job!

  However, some details can be improved:

  • The body's max-width property makes the site look awkward on larger resolutions.
  • Buttons' hover states are missing. Also, the font doesn't match the rest of the card.
  • Styling elements by their IDs is not a good practice. More info here.

  0

  This comment has received 1 replyShow