2011-02-17 Ojan Vafai <ojan@chromium.org>

        Reviewed by Adam Barth.

        create comments when hitting enter with a focused diff
        https://bugs.webkit.org/show_bug.cgi?id=54719

        * code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79164 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index e8ec48ef32e8b749a37eb96a76dc6ecea4749b7e..a2c523566c961f33970ac99a0229981f4795690c 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,12 @@
+2011-02-17  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        create comments when hitting enter with a focused diff
+        https://bugs.webkit.org/show_bug.cgi?id=54719
+
+        * code-review.js:
+
 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.

diff --git a/Websites/bugs.webkit.org/code-review.js b/Websites/bugs.webkit.org/code-review.js
index b92000bc1b47af034a03d6d75eb8f8923cae28cd..bbfeba5131765738c55cc12f7816cb00ec6dad05 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -331,12 +331,16 @@ var CODE_REVIEW_UNITTEST;
     else
       container.removeClass('saving');
   }
-
-  function createCommentFor(line) {
-    if (line.attr('data-has-comment')) {
+  
+  function unfreezeCommentFor(line) {
       // FIXME: This query is overly complex because we place comment blocks
       // after Lines.  Instead, comment blocks should be children of Lines.
       findCommentPositionFor(line).next().next().filter('.frozenComment').each(handleUnfreezeComment);
+  }
+
+  function createCommentFor(line) {
+    if (line.attr('data-has-comment')) {
+      unfreezeCommentFor(line);
       return;
     }
     line.attr('data-has-comment', 'true');
@@ -1453,7 +1457,15 @@ var CODE_REVIEW_UNITTEST;
       return true;
     }
 
-    return false;
+    var lines = $('.Line', focused);
+    var last = lines.last();
+    if (last.attr('data-has-comment')) {
+      unfreezeCommentFor(last);
+      return true;
+    }
+
+    addCommentForLines(lines);
+    return true;
   }
 
   function contextLinesFor(comment_base_lines, file_diff) {
@@ -1592,24 +1604,26 @@ var CODE_REVIEW_UNITTEST;
 
   function processSelectedLines() {
     drag_select_start_index = -1;
-
-    var selected = $('.selected');
-    if (!selected.size())
+    addCommentForLines($('.selected'));
+  }
+  
+  function addCommentForLines(lines) {    
+    if (!lines.size())
       return;
-    
-    var already_has_comment = selected.last().hasClass('commentContext');
-    selected.addClass('commentContext');
+
+    var already_has_comment = lines.last().hasClass('commentContext');
+    lines.addClass('commentContext');
 
     var comment_base_line;
     if (already_has_comment)
-      comment_base_line = selected.last().attr('data-comment-base-line');
+      comment_base_line = lines.last().attr('data-comment-base-line');
     else {
-      var last = lineFromLineDescendant(selected.last());
+      var last = lineFromLineDescendant(lines.last());
       addCommentFor($(last));
       comment_base_line = last.attr('id');
     }
 
-    selected.each(function() {
+    lines.each(function() {
       addDataCommentBaseLine(this, comment_base_line);
       $(this).removeClass('selected');
     });