Index: lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailProgressController.java =================================================================== diff -u -r328b2d5068eb7b9c271f43e55b3f19a05733312b -r1af0f74bf1b8d1e7dc2ee250022019478e6bb748 --- lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailProgressController.java (.../EmailProgressController.java) (revision 328b2d5068eb7b9c271f43e55b3f19a05733312b) +++ lams_monitoring/src/java/org/lamsfoundation/lams/monitoring/web/EmailProgressController.java (.../EmailProgressController.java) (revision 1af0f74bf1b8d1e7dc2ee250022019478e6bb748) @@ -148,8 +148,6 @@ } if (date != null) { dateJSON.put("id", date.getTime()); - dateJSON.put("ms", date.getTime()); - dateJSON.put("date", DateUtil.convertToStringForJSON(date, locale)); } return dateJSON; } Index: lams_monitoring/web/css/components-monitoring.css =================================================================== diff -u -r10df78ce6b0f3c72a5ab1d8e17510c08211c36f8 -r1af0f74bf1b8d1e7dc2ee250022019478e6bb748 --- lams_monitoring/web/css/components-monitoring.css (.../components-monitoring.css) (revision 10df78ce6b0f3c72a5ab1d8e17510c08211c36f8) +++ lams_monitoring/web/css/components-monitoring.css (.../components-monitoring.css) (revision 1af0f74bf1b8d1e7dc2ee250022019478e6bb748) @@ -997,7 +997,7 @@ width: 35px; height: 35px; line-height: 35px; - font-size: 13px; + font-size: 15px; border-radius: 20px; cursor: pointer; background-color: #e7e7e7; Index: lams_monitoring/web/includes/javascript/monitorLesson5.js =================================================================== diff -u -r10df78ce6b0f3c72a5ab1d8e17510c08211c36f8 -r1af0f74bf1b8d1e7dc2ee250022019478e6bb748 --- lams_monitoring/web/includes/javascript/monitorLesson5.js (.../monitorLesson5.js) (revision 10df78ce6b0f3c72a5ab1d8e17510c08211c36f8) +++ lams_monitoring/web/includes/javascript/monitorLesson5.js (.../monitorLesson5.js) (revision 1af0f74bf1b8d1e7dc2ee250022019478e6bb748) @@ -43,6 +43,10 @@ }, tempusDominusDateFormatter = function(date) { return date ? date.year + '-' + date.monthFormatted + '-' + date.dateFormatted + ' ' + date.hoursFormatted + ':' + date.minutesFormatted : ''; + } + dateFormatter = function(date) { + return ("0" + date.getDate()).slice(-2) + "-" + ("0"+(date.getMonth() + 1)).slice(-2) + "-" + + date.getFullYear() + " " + ("0" + date.getHours()).slice(-2) + ":" + ("0" + date.getMinutes()).slice(-2); }, COLORS = { @@ -257,6 +261,7 @@ var classDialog = showDialog('classDialog',{ 'autoOpen' : false, 'width' : 950, + 'height' : 700, 'title' : LABELS.LESSON_EDIT_CLASS, 'resizable' : true, 'close' : function(){ @@ -301,17 +306,21 @@ var emailProgressDialog = showDialog('emailProgressDialog',{ 'autoOpen' : false, - 'height' : 500, 'width' : 510, + 'height' : 700, 'title' : LABELS.PROGRESS_EMAIL_TITLE, - 'resizable' : false + 'resizable' : false, + 'close' : function(){} }, false); $('.modal-body', emailProgressDialog).empty().append($('#emailProgressDialogContents').show()); //initialize datetimepicker - new tempusDominus.TempusDominus(document.getElementById('emaildatePicker'), tempusDominusDefaultOptions) - .dates.formatInput = tempusDominusDateFormatter; + let datePickerElement = $('#emaildatePicker'), + datePicker = new tempusDominus.TempusDominus(datePickerElement[0], tempusDominusDefaultOptions); + datePicker.dates.formatInput = tempusDominusDateFormatter; + datePickerElement.data('datePicker', datePicker); + // sets gradebook on complete functionality $('#gradebookOnCompleteButton').change(function(){ var checked = $(this).prop('checked'), @@ -957,9 +966,9 @@ * Adds/removes date to the set of progress report emailing dates */ function editEmailProgressDate(dateCheckbox){ - var dateid = dateCheckbox.parent().attr('dateid'), - add = dateCheckbox.is(':checked'); - var data = { + var dateid = dateCheckbox.closest('.dialogListItem').attr('dateid'), + add = dateCheckbox.is(':checked'), + data = { 'lessonID' : lessonId, 'id' : dateid, 'add' : add @@ -971,9 +980,9 @@ cache : false, data : data, success : function( dateObj ) { - dateCheckbox.parent().attr('dateid', dateObj.id); - dateCheckbox.parent().attr('datems', dateObj.ms); - dateCheckbox.parent().children().last().html(dateObj.date); + dateCheckbox.closest('.dialogListItem') + .attr('dateid', dateObj.id) + .find('label').text(dateFormatter(new Date(dateObj.id))); } }); } @@ -1009,25 +1018,25 @@ $.ajax(ajaxProperties); $.each(dates, function(dateIndex, date) { - addCheckbox(date, list, true); + addCheckbox(date.id, list, true); }); } -function addCheckbox(dateObj, list, checked) { +function addCheckbox(dateId, list, checked) { // check for an existing matching date - var alreadyExists = false; - var existingDivs = $("div", list); + var alreadyExists = false, + existingDivs = $("tr", list); $.each(existingDivs, function(divIndex, div) { - if ( div.getAttribute('dateid') == dateObj.id ) { + if ( div.getAttribute('dateid') == dateId) { alreadyExists = true; return false; } }); if ( alreadyExists ) return; - + // does not exist so add to list - var checkboxId = 'email-progress-date-' + dateObj.id, + var checkboxId = 'email-progress-date-' + dateId, checkbox = $('').attr({ 'type' : 'checkbox', 'id' : checkboxId @@ -1036,17 +1045,19 @@ editEmailProgressDate($(this)); }), - dateString = $('').addClass('form-check-label').attr('for', checkboxId).text(dateObj.date), + + dateString = $('').addClass('form-check-label').attr('for', checkboxId).text(dateFormatter(new Date(dateId))), - dateDiv = $('
').attr({ - 'dateid' : dateObj.id, - 'datems' : dateObj.ms - }) - .addClass('dialogListItem') - .append(dateString) - .prepend(checkbox) - .appendTo(list); - + dateRow = $('