# HG changeset patch # User Steve Losh # Date 1278173251 14400 # Node ID c0e1c48cd6a85ab174067a19d49229f530beb220 # Parent 96d10fd456cff866758e440ffb2294cfebaabf99 web: whitespace diff -r 96d10fd456cf -r c0e1c48cd6a8 review/static/comments.js --- a/review/static/comments.js Sat Jul 03 12:05:03 2010 -0400 +++ b/review/static/comments.js Sat Jul 03 12:07:31 2010 -0400 @@ -14,15 +14,15 @@ '' + '' + - + '
' + '' + '' + '
' + - + 'Post Comment' + 'Cancel' + - + '' + '' + '' + @@ -37,33 +37,28 @@ } $(function() { - $(".activate a").click(function(event) { $(event.target).closest(".activate").hide(); $(event.target).closest("div").children("form").fadeIn("fast"); return false; }); - $(".activate-edit a").click(function(event) { $(event.target).closest(".activate-edit").hide(); $(event.target).closest(".comment").find(".message").hide(); $(event.target).closest("div").children("form").fadeIn("fast"); return false; }); - $("a.cancel").click(function(event) { $(event.target).closest(".togglebox").children(".activate").show(); $(event.target).parents("form").hide(); return false; }); - $("a.cancel-edit").click(function(event) { $(event.target).closest(".toggleinline").children(".activate-edit").show(); $(event.target).closest(".comment").find(".message").show(); $(event.target).parents("form").hide(); return false; }); - $("a.cancel-line").live('click', function(event) { $(event.target).closest(".diff").find(".chosen").removeClass("chosen"); $(event.target).closest("tr.comment-form").remove(); @@ -81,7 +76,6 @@ }); var lastSelected = null; - $(".commentable").click(function(event) { var currNum = parseInt($(this).find(".linenumber").html()); var diff = $(this).closest(".diff"); @@ -119,7 +113,7 @@ var existing_form_line_number = parseInt(existing_form.find(".lastlinenumber").html()); var existing_body = existing_form.find(".body").val(); var markdown = existing_form.find('.markdown-select').is(':checked'); - + if (existing_form_line_number != last_line_number) { existing_forms.remove(); @@ -140,6 +134,6 @@ return false; }); - + $("label.infield").inFieldLabels(); });