All comments

• 

  P

  Paul• 10

  @Chasusa

  Submitted almost 3 years ago

  This is my first submission on Frontend Mentor.

  Would appreciate feedback on my HTML structure and CSS as well. I feel that I wrote too much code and could maybe have simplified it better.

  Thanks for any feedback!

  

  Order Summary Card with CSS FLexbox

  4

  10

  

  Md Rafsan Jani Rafin• 70

  @itsrafsanjani

  Posted almost 3 years ago

  Your design looks perfect. But did you manually calculate those bottom, left, right, top? If yes then stop doing that. Try to use flex or grid as much as possible. So that other developers can read and modify your code easily. And use href="#" if you don't know the link.

  Marked as helpful

  0

  This comment has received 1 replyShow
• 

  Ravi Bhushan• 70

  @ravi-bhu

  Submitted almost 3 years ago

  Is there any improvement I can do here?

  

  Order Summary Component

  1

  10

  

  Md Rafsan Jani Rafin• 70

  @itsrafsanjani

  Posted almost 3 years ago

  body { background-repeat: no-repeat; background-size: cover; }

  Use this and also add hover state for the buttons.

  And also add some box-shadow for the card.

  Marked as helpful

  0

  This comment has received 1 replyShow