All comments

• 

  Joshua Early•30

  @JearlDevSubmitted almost 3 years ago

  Comments and criticisms are welcome :)

  

  ClockApp Using Vanilla JS and Axios

  4

  80
  

  Mo•130

  @mo-rob-1Posted almost 3 years ago

  Good job man, looks great! Only feedback is to remember to add alt text to the icon in your HTML to prevent accessibility issues.

  Keep up the good work :)

  0
• 

  Peter Hannell•200

  @peterhannellSubmitted almost 3 years ago

  My solution to this challenge. I'm not 100% satisfied with it but I think it's close.

  Any feedback and suggestions much appreciated.

  

  3-column Preview Card Component

  5

  50
  

  Mo•130

  @mo-rob-1Posted almost 3 years ago

  Good job dude, looks great! Also noticed in your code that you used the BEM technique and CSS custom properties.

  Keep up the good work :)

  Marked as helpful

  1
• 

  Larissa Cauane•70

  @larissacauaneSubmitted almost 3 years ago

  

  QR Code Component

  3

  20
  

  Mo•130

  @mo-rob-1Posted almost 3 years ago

  Hey! You did a great job on this challenge. Well done.

  After looking at your code, a tip I have is to have a look at the BEM methodology (Block Element Modifier) for naming your classes. This helps to keep your code organized and maintainable E.g. You could have a container div named 'card' and inside that div you can use naming conventions like 'card__img', 'card__desc' etc for the image and text.

  Hope this helps & keep up the good work :)

  Marked as helpful

  0