All comments

• 

  Ahmed Awad•80

  @AhmedAwad97Submitted about 1 year ago

  Hello, this is my first task i am a beginner, i will appreciate any feedback

  

  Responsive summary component using flexbox

  2

  61
  

  David Irvine•210

  @DavidIrvine-TWPosted about 1 year ago

  hi.. good job :)

  you should look at horizontal and vertical centering :)

  super easy using flexbox :)

  Marked as helpful

  1

  This comment has received 1 replyShow
• 

  SRI RAKESH V•590

  @srirakeshvSubmitted about 1 year ago

  All feedback are welcomed

  

  Intro Section Responsive

  1

  30
  

  David Irvine•210

  @DavidIrvine-TWPosted about 1 year ago

  at mid-screen sizes the image is very squashed...consider an appropriate image size. you can use the <picture> and <source> elements for that.

  otherwise good job! :)

  0

  This comment has received 1 replyShow
• 

  Jorge Sánchez•420

  @Jorge-sanchez09Submitted over 1 year ago

  • What did you find difficult while building the project?

  Tried to make it almost pixel perfect, so it took me about 1.5 hours adjusting every margin and padding

  • Which areas of your code are you unsure of?

  I'm not sure about the background image

  Any feedback is welcome

  

  Order summary component, practicing html and css

  1

  30
  

  David Irvine•210

  @DavidIrvine-TWPosted over 1 year ago

  looks great - I think you forgot to add a box shadow to the button tho. :)

  Also, the 'change' text is a little small. Otherwise, I think it's very close.

  well done! :)

  0