Merge pull request #70 from dingoblog/comment_js

Optimize front-end JS script

Author: dinever

view/default/assets/src/js/home.js

@@ -41,14 +41,17 @@ function initComment() {
             tpl.find(".comment-name").attr("href", json.comment.website).text(json.comment.author);
             tpl.find(".comment-reply").attr("rel", json.comment.id);
             tpl.find(".comment-content").html("<p>" + json.comment.content + "</p>");
-            var date = new Date(json.comment.create_time);
             tpl.find(".comment-message").html("Your comment is awaiting moderation.");
             tpl.attr("id", "comment-" + json.comment.id);
             if (json.comment.status == "approved") {
                 tpl.find(".comment-check").remove();
             }
             var parentId = $('#comment-parent').val();
-            tpl.insertAfter($('#comment-' + parentId));
+            if (parentId === '0') {
+                $('#comment-list').append(tpl);
+            } else {
+                tpl.insertAfter($('#comment-' + parentId));
+            }
             $('#comment-cancel').trigger("click");
             $('#comment-content').val("");
         } else {
@@ -62,7 +65,6 @@ function initComment() {
         form.removeClass("hide").insertAfter($(this));
         $('#comment-show').hide();
         $('#comment-parent').val(id);
-        $('#comment-form').removeClass("hide");
         $('.cancel-reply').show();
         var top = parentComment.offset().top;
         $('body,html').animate({scrollTop: top}, 500);

view/default/assets/src/js/navbar.js

@@ -2,14 +2,16 @@ $(document).ready(function () {
     $('#mobile-nav-trigger').click(function (el) {
         el.preventDefault();
 
+        var nav = $('#nav');
+
         if ($(this).hasClass('active')) {
             $(this).removeClass('active');
             $(this).html('Show Navigation');
-            $('#nav ul.nav').slideUp('fast').removeClass('active');
+            nav.find('ul.nav').slideUp('fast').removeClass('active');
         } else {
             $(this).addClass('active');
             $(this).html('Close Navigation');
-            $('#nav ul.nav').slideDown('fast').addClass('active');
+            nav.find('ul.nav').slideDown('fast').addClass('active');
         }
     });
 });