Index: lams_learning/src/java/org/lamsfoundation/lams/learning/discussion/controller/DiscussionSentimentController.java =================================================================== diff -u -r0d458a161ee50dd2db8283520b80b5cc559705bf -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_learning/src/java/org/lamsfoundation/lams/learning/discussion/controller/DiscussionSentimentController.java (.../DiscussionSentimentController.java) (revision 0d458a161ee50dd2db8283520b80b5cc559705bf) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/discussion/controller/DiscussionSentimentController.java (.../DiscussionSentimentController.java) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -135,7 +135,7 @@ int learnerCount = lessonService.getCountLessonLearners(lessonId, null); model.addAttribute("learnerCount", learnerCount); - return "/discussion/monitor"; + return "/discussion/monitor5"; } @RequestMapping(path = "/stopMonitor", method = RequestMethod.POST) Index: lams_learning/web/discussion/monitor5.jsp =================================================================== diff -u --- lams_learning/web/discussion/monitor5.jsp (revision 0) +++ lams_learning/web/discussion/monitor5.jsp (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -0,0 +1,297 @@ +<%@include file="/common/taglibs.jsp"%> + + + + + + + +
+
+
+ + + ( + + + + ) +
+
+ +
+
+ +
+
+ +
+
+
+
+
+ + + + + + + + + + + + + + + + + + + + + + +
+ + ( ) + +
+ + +
+ + ( ) + +
+ + +
+
+
+
+
\ No newline at end of file Index: lams_monitoring/web/css/components-monitoring.css =================================================================== diff -u -r56003dc7b87af7983c137daf04e402404932aeaa -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_monitoring/web/css/components-monitoring.css (.../components-monitoring.css) (revision 56003dc7b87af7983c137daf04e402404932aeaa) +++ lams_monitoring/web/css/components-monitoring.css (.../components-monitoring.css) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -1258,4 +1258,8 @@ .gate-opened-icon { cursor: pointer; +} + +.bg-success { + color: var(--bs-white) !important; } \ No newline at end of file Index: lams_monitoring/web/includes/javascript/monitorLesson5.js =================================================================== diff -u -r2a5fb646572b9790ef4b88e795e15811d94ed19d -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_monitoring/web/includes/javascript/monitorLesson5.js (.../monitorLesson5.js) (revision 2a5fb646572b9790ef4b88e795e15811d94ed19d) +++ lams_monitoring/web/includes/javascript/monitorLesson5.js (.../monitorLesson5.js) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -3106,11 +3106,16 @@ } function showConfirm(body, callback) { - let dialog = $('#confirmationDialog'); - $('.modal-body', dialog).html(body); + let dialog = $('#confirmationDialog').data('confirmed', null).off('hidden.bs.modal').on('hidden.bs.modal', function(){ + if (dialog.data('confirmed')) { + callback(true); + } + }); + + $('.modal-body', dialog).html(body) $("#confirmationDialogConfirmButton").off('click').on("click", function(){ - callback(true); + dialog.data('confirmed', true); dialog.modal('hide'); }); Index: lams_tool_assessment/web/pages/learning/results/allquestions5.jsp =================================================================== diff -u -r08f538ec5750238a01bc6b9a029c05531a201c2e -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_tool_assessment/web/pages/learning/results/allquestions5.jsp (.../allquestions5.jsp) (revision 08f538ec5750238a01bc6b9a029c05531a201c2e) +++ lams_tool_assessment/web/pages/learning/results/allquestions5.jsp (.../allquestions5.jsp) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -84,7 +84,7 @@ function startDiscussionSentiment(toolQuestionUid, burningQuestionUid, markAsActive) { // burningQuestionUid is not used in Assessment, but must be added for function signature consistency - $('#discussion-sentiment-chart-panel-container-' + toolQuestionUid).load( + $('#discussion-sentiment-chart-card-container-' + toolQuestionUid).load( '${lams}learning/discussionSentiment/startMonitor.do', { toolQuestionUid : toolQuestionUid, @@ -149,20 +149,20 @@
-
+
<%-- Allow disclosing correct answers only for multiple choice questions --%> -
- disabled="disabled">   >
-
- disabled="disabled">   > @@ -171,17 +171,17 @@ -
+
-  
 
@@ -323,7 +323,7 @@
-
+
Index: lams_tool_assessment/web/pages/monitoring/parts/discloseAnswers5.jsp =================================================================== diff -u -r08f538ec5750238a01bc6b9a029c05531a201c2e -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_tool_assessment/web/pages/monitoring/parts/discloseAnswers5.jsp (.../discloseAnswers5.jsp) (revision 08f538ec5750238a01bc6b9a029c05531a201c2e) +++ lams_tool_assessment/web/pages/monitoring/parts/discloseAnswers5.jsp (.../discloseAnswers5.jsp) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -24,19 +24,18 @@ button.click(function(event) { - if (!confirm(isCorrectButton ? "" - : "")) { - return; - } - - // check if correct answers are disclosed before groups' answers - if (isCorrectButton && - !button.closest('.disclose-button-group').find('.disclose-groups-button').is('[disabled]') && - !confirm("")) { - return; - } - - discloseAnswers(button, resultsPane, true); + showConfirm(isCorrectButton ? "" + : "", function(){ + // check if correct answers are disclosed before groups' answers + if (isCorrectButton && + !button.closest('.disclose-button-group').find('.disclose-groups-button').is('[disabled]')) { + showConfirm("", function(){ + discloseAnswers(button, resultsPane, true); + }); + } else { + discloseAnswers(button, resultsPane, true); + } + }); }); }); @@ -47,23 +46,25 @@ // do not add a handler twice if (!isRefresh) { allCorrectButton.click(function(){ - if (!confirm("")) { - return; - } - - // check if correct answers are disclosed before groups' answers - if (!$('.disclose-all-groups-button', assessmentQuestionsPane).is('[disabled]') && - !confirm("")) { - return; - } - - let nonDisclosedQuestions = $('.disclose-correct-button', assessmentQuestionsPane).not('[disabled]'), - lastQuestionUid = nonDisclosedQuestions.last().closest('.disclose-button-group').attr('questionUid'); - nonDisclosedQuestions.each(function() { - let isLast = lastQuestionUid == $(this).closest('.disclose-button-group').attr('questionUid'); - discloseAnswers($(this), resultsPane, isLast); + showConfirm("", function(){ + // check if correct answers are disclosed before groups' answers + let discloseFunction = function(){ + let nonDisclosedQuestions = $('.disclose-correct-button', assessmentQuestionsPane).not('[disabled]'), + lastQuestionUid = nonDisclosedQuestions.last().closest('.disclose-button-group').attr('questionUid'); + nonDisclosedQuestions.each(function() { + let isLast = lastQuestionUid == $(this).closest('.disclose-button-group').attr('questionUid'); + discloseAnswers($(this), resultsPane, isLast); + }); + disabledAndCheckButton(allCorrectButton); + }; + + if (!$('.disclose-all-groups-button', assessmentQuestionsPane).is('[disabled]')) { + showConfirm("", discloseFunction); + return; + } + + discloseFunction(); }); - disabledAndCheckButton(allCorrectButton); }); } } else { @@ -75,17 +76,15 @@ // do not add a handler twice if (!isRefresh) { allGroupsButton.click(function(){ - if (!confirm("")) { - return; - } - - let nonDisclosedQuestions = $('.disclose-groups-button', assessmentQuestionsPane).not('[disabled]'), - lastQuestionUid = nonDisclosedQuestions.last().closest('.disclose-button-group').attr('questionUid'); - nonDisclosedQuestions.each(function() { - let isLast = lastQuestionUid == $(this).closest('.disclose-button-group').attr('questionUid'); - discloseAnswers($(this), resultsPane, isLast); + showConfirm("", function(){ + let nonDisclosedQuestions = $('.disclose-groups-button', assessmentQuestionsPane).not('[disabled]'), + lastQuestionUid = nonDisclosedQuestions.last().closest('.disclose-button-group').attr('questionUid'); + nonDisclosedQuestions.each(function() { + let isLast = lastQuestionUid == $(this).closest('.disclose-button-group').attr('questionUid'); + discloseAnswers($(this), resultsPane, isLast); + }); + disabledAndCheckButton(allGroupsButton); }); - disabledAndCheckButton(allGroupsButton); }); } } else { @@ -97,7 +96,6 @@ function discloseAnswers(button, resultsPane, isLast) { let isCorrectButton = button.hasClass('disclose-correct-button'), toolContentId = resultsPane.data('toolContentId'); - $.ajax({ 'url' : 'monitoring/' + (isCorrectButton ? 'discloseCorrectAnswers' : 'discloseGroupsAnswers') @@ -116,6 +114,6 @@ } function disabledAndCheckButton(button){ - button.attr('disabled', true).html(' ' + button.text()); + button.attr('disabled', true).html(' ' + button.text()); } \ No newline at end of file Index: lams_tool_assessment/web/pages/tblmonitoring/assessment5.jsp =================================================================== diff -u -r2a5fb646572b9790ef4b88e795e15811d94ed19d -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_tool_assessment/web/pages/tblmonitoring/assessment5.jsp (.../assessment5.jsp) (revision 2a5fb646572b9790ef4b88e795e15811d94ed19d) +++ lams_tool_assessment/web/pages/tblmonitoring/assessment5.jsp (.../assessment5.jsp) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -26,52 +26,50 @@ } -
- <%-- For AEs tab the panes are defined in TBL monitor, for IRA we need to define it here --%> -
- -
-
-

- -

-
+<%-- For AEs tab the panes are defined in TBL monitor, for IRA we need to define it here --%> +
+ +
+
+

+ +

- - - -
-
-

- - : ${attemptedLearnersNumber}/ -

-
-
-
- - -
-
- - - <%-- Release correct/groups answers for all questions in this assessment --%> -
-
-
-
- -
-
- -
-
-
-
-
+ -
+ +
+
+

+ + : ${attemptedLearnersNumber}/ +

+
+
+ +
-
\ No newline at end of file + + + <%-- Release correct/groups answers for all questions in this assessment --%> +
+
+ +
+
+ +
+
+
+ +
+
Index: lams_tool_assessment/web/pages/tblmonitoring/assessmentStudentChoices5.jsp =================================================================== diff -u -r2a5fb646572b9790ef4b88e795e15811d94ed19d -rac1bc5dc42645586cf573699676b6f73fcf59d09 --- lams_tool_assessment/web/pages/tblmonitoring/assessmentStudentChoices5.jsp (.../assessmentStudentChoices5.jsp) (revision 2a5fb646572b9790ef4b88e795e15811d94ed19d) +++ lams_tool_assessment/web/pages/tblmonitoring/assessmentStudentChoices5.jsp (.../assessmentStudentChoices5.jsp) (revision ac1bc5dc42645586cf573699676b6f73fcf59d09) @@ -155,7 +155,7 @@ - bg-success text-white" > + bg-success" > ${questionResultDto.answer}