All comments

• 

  Ahmed Awad•80

  @AhmedAwad97

  Submitted almost 2 years ago

  

  Responsive summary component using flexbox

  #web-components

  2

  61
  

  David Irvine•210

  @DavidIrvine-TW

  Posted almost 2 years ago

  hi.. good job :)

  you should look at horizontal and vertical centering :)

  super easy using flexbox :)

  1

  Marked as helpful
• 

  SRI RAKESH V•590

  @srirakeshv

  Submitted almost 2 years ago

  

  Intro Section Responsive

  1

  30
  

  David Irvine•210

  @DavidIrvine-TW

  Posted almost 2 years 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
• 

  Jorge Sánchez•420

  @Jorge-sanchez09

  Submitted about 2 years ago

  

  Order summary component, practicing html and css

  #bem

  1

  30
  

  David Irvine•210

  @DavidIrvine-TW

  Posted about 2 years 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