3 ms·

The lack of semicolons and the use of jshint bothers me.

• tinco 14y ago
  The fact that the code is very neat and is well documented bothers me. Also the fact that javascript is a lot more readable when you put a blank line between every two lines bothers me (a lot!). And that it works perfectly and looks great, man that bothers me.
  • devinus 14y ago
    // If datePassed is true else if ( datePassed === true ) { Yes, very well documented. >.>
    • thezilch 14y ago
      Indeed. In fact, I'd posture that the comments, after the first, are wrong; the lines' intents are to _set_! // Set the element as readonly element.readOnly = true // Get the date today DATE_TODAY = P.getDateToday() // Get the date to select DATE_SELECTED = P.getDateSelected() // Get the month to focus MONTH_FOCUSED = P.getMonthFocused() // Get the date ranges DATE_MIN = P.getDateRange( SETTINGS.date_min ) DATE_MAX = P.getDateRange( SETTINGS.date_max, 1 )
    • Stratoscope 14y ago
      // Return the calendarObject return calendarObject It's like this all the way through the code!
• insin 14y ago
  It doesn't lack semicolons, it's only using them where necessary.