mobile-friendly advice generator app with fetch, scss and flexbox

Community feedback

• 

  Fazza Razaq Amiarso•2,340

  @fazzaamiarsoPosted over 2 years ago

  Hello LuckyChimp! Nice solution overall!

  I just have a quick tips for concatenating string. You can use template literal. This will make your work easier. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Template_literals

  //instead of this
  adviceTextElmnt.textContent = '"'+adviceText+'"';
  
  //to 
  adviceTextElmnt.textContent = `"${adviceText}"`;
  
  

  I hope it helps! Cheers!

  Marked as helpful

  1

  ---

  

  LuckyChimp•160

  @LuckyChimpPosted over 2 years ago

  @fazzaamiarso Thanks for pointing that out! This is really easier and especially with complex chains also nicer to read, I think. I'll try to use template literals only in the future. Again, thank you! :)

  0