piatra/date-picker — comment on commit 04c355928cb167cb402deca91450262548fb4c02
File:  line 0
By calvinbushor:

Thank you for you commit, I have some feedback before accepting.

In the constructor you declare var t = Date(). Please create a property rather than a variable: this.date = new Date(); You can reference that date for your current properties like this; currentDay = this.date.getDay().

I noticed you used different syntax than us. We would prefer you use opening and closing brackets ( { ) in your conditionals. There are several other syntax variances but please try to keep the syntax consistent because now the code has mixed coding practices.

We would like to use the class name of dp_current instead of time_today.

Please remove all of your console.logs();

I think that is it.