Commit 1a29c212 by Xavier Antoviaque

Merge pull request #73 from edx-solutions/dcikatic/MCKIN-2175-feedback-popup-fix

MCKIN-2175 Remove popup html on clearResult call and add popup text only...
parents a09b6fdc ecf34eb1
...@@ -53,7 +53,7 @@ function MessageView(element, mentoring) { ...@@ -53,7 +53,7 @@ function MessageView(element, mentoring) {
} }
}, },
clearResult: function() { clearResult: function() {
this.allPopupsDOM.hide(); this.allPopupsDOM.html('').hide();
this.allResultsDOM.removeClass( this.allResultsDOM.removeClass(
'checkmark-incorrect icon-exclamation fa-exclamation checkmark-correct icon-ok fa-check' 'checkmark-incorrect icon-exclamation fa-exclamation checkmark-correct icon-ok fa-check'
); );
...@@ -101,7 +101,7 @@ function MCQBlock(runtime, element, mentoring) { ...@@ -101,7 +101,7 @@ function MCQBlock(runtime, element, mentoring) {
choiceResultDOM.addClass('checkmark-incorrect icon-exclamation fa-exclamation'); choiceResultDOM.addClass('checkmark-incorrect icon-exclamation fa-exclamation');
} }
if (result.tips) { if (result.tips && choiceInputDOM.val() === result.submission) {
mentoring.setContent(choiceTipsDOM, result.tips); mentoring.setContent(choiceTipsDOM, result.tips);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment