misc comments

jshint.conf.js

@@ -3,9 +3,10 @@ module.exports = {
 	options: {
 		browser: true,
 		globals: {
-			define: true,
-			moment: true,
-			jQuery: true
+			// `false` means read-only
+			define: false,
+			moment: false,
+			jQuery: false
 		},
 		bitwise: true,
 		camelcase: true,

src/Calendar.js

@@ -709,6 +709,7 @@ function Calendar(element, instanceOptions) {
 	------------------------------------------------------------------------*/
 	
 	if (options.droppable) {
+		// TODO: unbind on destroy
 		$(document)
 			.bind('dragstart', function(ev, ui) {
 				var _e = ev.target;

src/common/SelectionManager.js

@@ -28,6 +28,7 @@ function SelectionManager() {
 
 	// unselectAuto
 	if (opt('selectable') && opt('unselectAuto')) {
+		// TODO: unbind on destroy
 		$(document).mousedown(function(ev) {
 			var ignore = opt('unselectCancel');
 			if (ignore) {

src/moment-ext.js

@@ -52,6 +52,7 @@ function makeMoment(args, parseUTC, parseZone) {
 		isAmbigZone = true;
 	}
 
+	// instantiate a vanilla moment
 	if (parseUTC || parseZone || isAmbigTime) {
 		mom = moment.utc.apply(moment, args);
 	}
@@ -96,6 +97,7 @@ function FCMoment(config) {
 // chain the prototype to Moment's
 FCMoment.prototype = createObject(moment.fn);
 
+// we need this because Moment's implementation will not copy of the ambig flags
 FCMoment.prototype.clone = function() {
 	return makeMoment([ this ]);
 };