Index: lams_central/web/WEB-INF/tags/Comments.tag =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/WEB-INF/tags/Comments.tag (.../Comments.tag) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/WEB-INF/tags/Comments.tag (.../Comments.tag) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -18,7 +18,7 @@ - + @@ -34,19 +34,14 @@ - - - - - - +
\ No newline at end of file Index: lams_central/web/comments/comments.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/comments.jsp (.../comments.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/comments.jsp (.../comments.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -42,27 +42,25 @@ scrollDoneCallback(); $('#sortMenu').change(function(){ - var url = "/comments/viewTopic.do?sessionMapID=${sessionMapID}&sortBy="+$(this).find("option:selected").attr('value'); + var url = "comments/viewTopic.do?sessionMapID=${sessionMapID}&sortBy="+$(this).find("option:selected").attr('value'); reloadScroll(url); }); }); function refreshComments(){ var reqIDVar = new Date(); - reloadScroll('/comments/viewTopic.do?sessionMapID=${sessionMapID}&reqUid='+reqIDVar.getTime()); + reloadScroll('comments/viewTopic.do?sessionMapID=${sessionMapID}&reqUid='+reqIDVar.getTime()); } function reloadScroll(url) { $('#newcomments').children().remove(); $('.scroll').load(url, function() { $('.scroll').data('jscroll', null); $('.scroll').jscroll({loadingHtml: '${loading_words}${loading_words}',padding:30,autoTrigger:false,callback:scrollDoneCallback}); - resizeIframe(); }); } function scrollDoneCallback() { - resizeIframe(); } Index: lams_central/web/comments/edit.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/edit.jsp (.../edit.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/edit.jsp (.../edit.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -45,7 +45,7 @@ // make sure the old edit form is gone, so the user won't try to submit it again $('#edit').remove(); } - reloadThread(response); + reloadThread(response,'','',''); }); } // end validateBodyText return false; @@ -58,7 +58,7 @@
-
"> + Index: lams_central/web/comments/msgview.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/msgview.jsp (.../msgview.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/msgview.jsp (.../msgview.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -54,17 +54,13 @@ - - - + comments/newReplyTopic.do?sessionMapID=${sessionMapID}&parentUid=${commentDto.comment.uid} Reply · - - - + comments/editTopic.do?sessionMapID=${sessionMapID}&commentUid=${commentDto.comment.uid}&create=${commentDto.comment.created.time} Edit · @@ -74,16 +70,12 @@ - - - + comments/hide.do?sessionMapID=${sessionMapID}&commentUid=${commentDto.comment.uid}&hideFlag=false - - - + comments/hide.do?sessionMapID=${sessionMapID}&commentUid=${commentDto.comment.uid}&hideFlag=true" Index: lams_central/web/comments/new.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/new.jsp (.../new.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/new.jsp (.../new.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -40,10 +40,9 @@ if ( ! newCommentDiv || ! newThreadDiv ) { alert(''); } else if ( commentUid ) { - var loadString = '' + response.sessionMapID + "&threadUid=" + threadUid+"&commentUid="+commentUid; + var loadString = 'comments/viewTopicThread.do?sessionMapID=' + response.sessionMapID + "&threadUid=" + threadUid+"&commentUid="+commentUid; $(newThreadDiv).load(loadString, function() { highlightMessage(); - resizeIframe(); }); clearNewForm(); } else if ( response.errMessage ) { @@ -65,7 +64,7 @@
-"> + Index: lams_central/web/comments/reply.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/reply.jsp (.../reply.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/reply.jsp (.../reply.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -44,7 +44,7 @@ // the user won't try to submit it again $('#reply').remove(); } - reloadThread(response); + reloadThread(response,'','',''); }); } // end validateBodyText return false; @@ -57,7 +57,7 @@
-"> + Index: lams_central/web/comments/topicview.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/topicview.jsp (.../topicview.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/topicview.jsp (.../topicview.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -42,9 +42,7 @@ var parentDiv = document.getElementById('msg'+commentUid); parentDiv.appendChild(replyDiv); - $(replyDiv).load(url, function() { - resizeIframe(); - }); + $(replyDiv).load(url); } } @@ -61,9 +59,7 @@ var parentDiv = document.getElementById('msg'+commentUid); parentDiv.appendChild(editDiv); - $(editDiv).load(url, function() { - resizeIframe(); - }); + $(editDiv).load(url); } } @@ -74,7 +70,7 @@ url: url }) .done(function (response) { - reloadThread(response); + reloadThread(response, '','',''); }); } @@ -98,7 +94,7 @@ function likeEntry(commentUid) { updateLike(commentUid, - '?sessionMapID=${sessionMapID}&commentUid='+commentUid, + 'comments/like.do?sessionMapID=${sessionMapID}&commentUid='+commentUid, 1); $( '#msglikebutton'+commentUid ).removeClass( 'fa-thumbs-up fa-faded' ).addClass( 'fa-thumbs-o-up' ); $( '#msglikebutton'+commentUid ).prop( 'onclick', null ); @@ -110,7 +106,7 @@ function dislikeEntry(commentUid) { updateLike(commentUid, - '?sessionMapID=${sessionMapID}&commentUid='+commentUid, + 'comments/dislike.do?sessionMapID=${sessionMapID}&commentUid='+commentUid, -1); $( '#msgdislikebutton'+commentUid ).removeClass( 'fa-thumbs-down fa-faded' ).addClass( 'fa-thumbs-o-down' ); $( '#msgdislikebutton'+commentUid ).prop( 'onclick', null ); @@ -169,7 +165,7 @@
- + /comments/viewTopic.do?pageLastId=${maxThreadUid}&likeCount=${minThreadLike}&pageSize=${sessionMap.pageSize}&sessionMapID=${sessionMapID} " class="button">
Index: lams_central/web/comments/topicviewwrapper.jsp =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/comments/topicviewwrapper.jsp (.../topicviewwrapper.jsp) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/comments/topicviewwrapper.jsp (.../topicviewwrapper.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -12,7 +12,7 @@ - + <%@ include file="topicview.jsp"%> Index: lams_central/web/includes/javascript/comments.js =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_central/web/includes/javascript/comments.js (.../comments.js) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_central/web/includes/javascript/comments.js (.../comments.js) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -1,7 +1,3 @@ - function resizeIframe() { - parent.resizeCommentFrame(document.body.scrollHeight); - } - function highlightMessage() { $('.highlight').filter($('table')).css('background','none'); $('.highlight').filter($('div')).effect('highlight', {color: "#fcf0ad"}, 6000); @@ -18,16 +14,17 @@ } } - function reloadThread(response) { + + function reloadThread(response, lamsUrl, savedCannotRedisplayMessage, errorCannotRedisplayMessage) { var threadDiv = document.getElementById('thread'+response.threadUid); var threadUid = response.threadUid; var commentUid = response.commentUid; if ( commentUid ) { if ( ! threadDiv) { - alert(''); + alert(savedCannotRedisplayMessage); } else { - var loadString = "viewTopicThread.do?&sessionMapID=" + response.sessionMapID + "&threadUid=" + threadUid+"&commentUid="+commentUid; + var loadString = lamsUrl+"comments/viewTopicThread.do?&sessionMapID=" + response.sessionMapID + "&threadUid=" + threadUid+"&commentUid="+commentUid; $(threadDiv).load(loadString, function() { // expand up to the reply - in case it is buried down in a lot of replies // don't need to do this if we have started a new thread. @@ -36,13 +33,12 @@ $('#msg'+commentUid).focus(); } highlightMessage(); - resizeIframe(); }); } } else if ( response.errMessage ) { // No valid id? Something failed. Assume it is a response message coming back. alert(response.errMessage); } else { - alert(''); + alert(errorCannotRedisplayMessage); } } Index: lams_tool_nb/web/WEB-INF/tags/Comments.tag =================================================================== diff -u -rd3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746 -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_tool_nb/web/WEB-INF/tags/Comments.tag (.../Comments.tag) (revision d3c9d3f03c16e8f7025d7b2bfe84e68af4fc0746) +++ lams_tool_nb/web/WEB-INF/tags/Comments.tag (.../Comments.tag) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -18,7 +18,7 @@
- + @@ -38,11 +38,10 @@ - - +
\ No newline at end of file Index: lams_tool_nb/web/template/learnerTemplate.jsp =================================================================== diff -u -r18e5e690d2e1a7c3d918759f7a2827c8205feaac -r5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8 --- lams_tool_nb/web/template/learnerTemplate.jsp (.../learnerTemplate.jsp) (revision 18e5e690d2e1a7c3d918759f7a2827c8205feaac) +++ lams_tool_nb/web/template/learnerTemplate.jsp (.../learnerTemplate.jsp) (revision 5297fc67fa56d2984dfa874443ebcf7ddc6a0ca8) @@ -14,6 +14,7 @@ <fmt:message key="activity.title"/> +