Vanilla JS heavy project

Community feedback

• 

  Elaine•11,400

  @elaineleungPosted over 2 years ago

  Hi Tbrownlee, I think you did a great job using async/await and also in creating divs for the data! 🙂 Also I think while your solution is slightly smaller than the design, it does look fairly close to the original, so maybe if you increase the font size and some padding, you'd be fairly spot on.

  I think as a further challenge, you can see whether you can make the code more DRY because I do think it's possible. Since you are now well able to use for loops, you can also try using forEach for iteration also.

  Once again, well done, and hope to see more 😊

  Marked as helpful

  1

  ---

  

  tbrownlee•350

  @tbrownleePosted over 2 years ago

  @elaineleung thank you for your helpful advice!

  My projects do tend to lean towards the smaller side, I'll try to make them more accurate.

  I will fix my render methods to make them more DRY since they basically use the same code.

  I will implement forEach in my projects more often in the future.

  0