All comments

• 

  victor• 40

  @victorgfz

  Submitted over 1 year ago

  It took a little longer than i expected, i found difficult on making the validation and i'm not sure i made it in the best way, since i used an input type="text" on email to avoid the default message. Feedbacks are welcome.

  

  Responsive layout using flex, form validation with js

  1

  51

  

  Handyyweb• 120

  @HandyyWeb

  Posted over 1 year ago

  Your code is very good, well done! Just a small thing that can help you, but it's not really necessary : Instead of setting margin : 0 or padding : 0 for each element, you can maybe use CSS Reset

  Personally, I use :

  *{
  padding : 0
  margin  :  0
  box-size : border-box
  }
  

  But choose what seems to be good for you, as I said it can even be skipped. Once again, your code is very good ! Have a nice day !

  Marked as helpful

  0
• 

  GreXLin85• 80

  @GreXLin85

  Submitted over 1 year ago

  

  Product Preview Card Component made with Flexbox

  1

  30

  

  Handyyweb• 120

  @HandyyWeb

  Posted over 1 year ago

  I think you can improve the responsive layout by using the <picture> and <source> elements to change the image source in relation to the viewport width.

  Here is a link if you need more information: HTML <picture> element.

  Above all, your code is great, well done!

  Marked as helpful

  1