Added some todos

pull/4/head
Mark McDowall 12 years ago
parent a6a5277de8
commit 1f03888ff9

@ -14,7 +14,7 @@
//this.validators.allowed = function (value) { //this.validators.allowed = function (value) {
// return value.length > 0 ? { isValid: true } : { isValid: false, message: 'You must have allowed qualities' }; // return value.length > 0 ? { isValid: true } : { isValid: false, message: 'You must have allowed qualities' };
//}; //};
//Todo: Cutoff should be something that is allowed (double check)
this.validators.cutoff = function (value) { this.validators.cutoff = function (value) {
return value != null ? { isValid: true } : { isValid: false, message: 'You must have a valid cutoff' }; return value != null ? { isValid: true } : { isValid: false, message: 'You must have a valid cutoff' };
}; };

@ -37,6 +37,11 @@
else { else {
$(cutoff).find('option[value="' + qualityId + '"]').remove(); $(cutoff).find('option[value="' + qualityId + '"]').remove();
} }
//Todo: auto-sort by weight (which is not the value)
//$(cutoff).html($($(cutoff).children("option")).sort(function(a, b) {
// return a.value == b.value ? 0 : a.value < b.value ? -1 : 1;
//}));
} }
}); });

Loading…
Cancel
Save