All comments

• 

  Asif Ali Khan• 210

  @asifkhan0410

  Submitted over 3 years ago

  I would love to know your feedback on how I can optimize it. If you have any suggestions do mention them in the comments. I would love to hear from you.

  

  Use CSS flexbox and Vanilla Js

  1

  1

  0

  

  VIDIT SHAH• 15

  @tensor0210

  Posted over 3 years ago

  Hey! Great job Code really documented well Few Code Improvements which will help when collaborating with others:

  1- File name: main.css and script.js instead of index.css and index.css 2- Use const for all Document objects(apart from sliders) 3-Avoid var 4-Don't use console.log in the production 5-DRY - Don't repeat yourself Eg pricing.innerHTML = "$" + 8 pageviews.innerHTML = 10+"K" convert this into a function 6-Make use of more string Literals

  Again great Job mate! Learnt a lot of CSS from your code base. Cheers :D!

  1

  This comment has received 1 replyShow