2011-02-01 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Feb 2011 05:17:38 +0000 (05:17 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 2 Feb 2011 05:17:38 +0000 (05:17 +0000)
        Reviewed by Adam Barth.

        avoid jitter when loading the comments to a patch
        https://bugs.webkit.org/show_bug.cgi?id=53570

        * code-review.js:

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

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

index 9f01a70..6dc492f 100644 (file)
@@ -1,5 +1,14 @@
 2011-02-01  Ojan Vafai  <ojan@chromium.org>
 
+        Reviewed by Adam Barth.
+
+        avoid jitter when loading the comments to a patch
+        https://bugs.webkit.org/show_bug.cgi?id=53570
+
+        * code-review.js:
+
+2011-02-01  Ojan Vafai  <ojan@chromium.org>
+
         Reviewed by Mihai Parparita.
 
         [codereviewtool] extra space at the beginning of lines in side-by-side view
index 553dded..ba10936 100644 (file)
@@ -417,7 +417,7 @@ body {
 }
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
-<script src="code-review.js?version=31"></script>
+<script src="code-review.js?version=32"></script>
 EOF
 
     def self.revisionOrDescription(string)
index d626d3a..f5f4b44 100644 (file)
@@ -374,16 +374,14 @@ var CODE_REVIEW_UNITTEST;
       });
     }
 
-    var help_text = 'Scroll though diffs with the "j" and "k" keys.';
     if (comments.length == 0) {
-      $('#message .commentStatus').text(help_text);
       return;
     }
 
     descriptor = comments.length + ' comment';
     if (comments.length > 1)
       descriptor += 's';
-    $('#message .commentStatus').text('This patch has ' + descriptor + '.  Scroll through them with the "n" and "p" keys. ' + help_text);
+    $('.help').append(' This patch has ' + descriptor + '.  Scroll through them with the "n" and "p" keys.');
   }
 
   function scanForStyleQueueComments(text) {
@@ -997,10 +995,9 @@ var CODE_REVIEW_UNITTEST;
     crawlDiff();
     fetchHistory();
     $(document.body).prepend('<div id="message">' +
-        '<div class="help">Select line numbers to add a comment.' +
+        '<div class="help">Select line numbers to add a comment. Scroll though diffs with the "j" and "k" keys.' +
           '<div class="DiffLinks LinkContainer">' + diffLinksHtml() + '</div>' +
         '</div>' +
-        '<div class="commentStatus"></div>' +
         '</div>');
     $(document.body).append('<div id="toolbar">' +
         '<div class="overallComments">' +