2011-02-21 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Feb 2011 03:59:52 +0000 (03:59 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Feb 2011 03:59:52 +0000 (03:59 +0000)
        Reviewed by Adam Barth.

        [codereviewtool] focus comment on accept
        https://bugs.webkit.org/show_bug.cgi?id=54930

        * code-review.js:

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

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js

index b682348..0746ba2 100644 (file)
@@ -1,3 +1,12 @@
+2011-02-21  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        [codereviewtool] focus comment on accept
+        https://bugs.webkit.org/show_bug.cgi?id=54930
+
+        * code-review.js:
+
 2011-02-20  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.
index b391745..00efb36 100644 (file)
@@ -1352,7 +1352,8 @@ var CODE_REVIEW_UNITTEST;
   }
   
   function acceptComment(comment) {
-    freezeComment(comment);
+    var frozen_comment = freezeComment(comment);
+    focusOn(frozen_comment);
     saveDraftComments();
   }
 
@@ -1375,7 +1376,9 @@ var CODE_REVIEW_UNITTEST;
     }
     var line_id = comment_textarea.attr('data-comment-for');
     var line = $('#' + line_id)
-    findCommentBlockFor(line).hide().after($('<div class="frozenComment"></div>').text(comment_textarea.val()));
+    var frozen_comment = $('<div class="frozenComment"></div>').text(comment_textarea.val());
+    findCommentBlockFor(line).hide().after(frozen_comment);
+    return frozen_comment;
   }
 
   function focusOn(node) {