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

        Reviewed by Adam Barth.

        use shift+j/k to focus next/previous line
        https://bugs.webkit.org/show_bug.cgi?id=54723

        Tested on a large Hyatt change to confirm it performed fine.
        (https://bugs.webkit.org/attachment.cgi?id=82533&action=review)

        * code-review.js:

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

diff --git a/Websites/bugs.webkit.org/ChangeLog b/Websites/bugs.webkit.org/ChangeLog
index 62936c57056990a7340d47db7d878cb75feb7574..be0267cab0a0af3b12573b3218012d8f92668dc6 100644 (file)
--- a/Websites/bugs.webkit.org/ChangeLog
+++ b/Websites/bugs.webkit.org/ChangeLog
@@ -1,3 +1,15 @@
+2011-02-17  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
+        use shift+j/k to focus next/previous line
+        https://bugs.webkit.org/show_bug.cgi?id=54723
+
+        Tested on a large Hyatt change to confirm it performed fine.
+        (https://bugs.webkit.org/attachment.cgi?id=82533&action=review)
+
+        * code-review.js:
+
 2011-02-17  Ojan Vafai  <ojan@chromium.org>
 
         Reviewed by Adam Barth.

diff --git a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
index 1718268b12a4c9c2551177f91e5a9f8aad7f84c3..cb88130d7b817eb0d193d80c6da3175ba2d1d183 100644 (file)
--- a/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
+++ b/Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb
@@ -447,7 +447,7 @@ div:focus {
 }
 </style>
 <script src="https://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js"></script> 
-<script src="code-review.js?version=35"></script>
+<script src="code-review.js?version=36"></script>
 EOF
 
     def self.revisionOrDescription(string)

diff --git a/Websites/bugs.webkit.org/code-review.js b/Websites/bugs.webkit.org/code-review.js
index 10651884dc3de66cdf97aa2296bc25004e3d18e9..d31adf3bf9f8e15bf35d8537ae387e73148a48b5 100644 (file)
--- a/Websites/bugs.webkit.org/code-review.js
+++ b/Websites/bugs.webkit.org/code-review.js
@@ -1349,7 +1349,7 @@ var CODE_REVIEW_UNITTEST;
   }
 
   function focusNext(filter, direction) {
-    var focusable_nodes = $('a,.frozenComment,.previousComment,.DiffBlock,.overallComments').filter(function() {
+    var focusable_nodes = $('a,.Line,.frozenComment,.previousComment,.DiffBlock,.overallComments').filter(function() {
       return !$(this).hasClass('DiffBlock') || $('.add,.remove', this).size();
     });
 
@@ -1376,6 +1376,8 @@ var CODE_REVIEW_UNITTEST;
   var kCharCodeForP = 'p'.charCodeAt(0);
   var kCharCodeForJ = 'j'.charCodeAt(0);
   var kCharCodeForK = 'k'.charCodeAt(0);
+  var kCharCodeForCapitalJ = 'J'.charCodeAt(0);
+  var kCharCodeForCapitalK = 'K'.charCodeAt(0);
   var kCharCodeForEnter = '\r'.charCodeAt(0);
 
   function isComment(node) {
@@ -1385,6 +1387,10 @@ var CODE_REVIEW_UNITTEST;
   function isDiffBlock(node) {
     return node.hasClass('DiffBlock');
   }
+  
+  function isLine(node) {
+    return node.hasClass('Line');
+  }
 
   $('textarea').live('keydown', function(e) {
     var escape_key_code = 27;
@@ -1416,6 +1422,14 @@ var CODE_REVIEW_UNITTEST;
     case kCharCodeForK:
       handled = focusNext(isDiffBlock, DIRECTION.BACKWARD);
       break;
+      
+    case kCharCodeForCapitalJ:
+      handled = focusNext(isLine, DIRECTION.FORWARD);
+      break;
+
+    case kCharCodeForCapitalK:
+      handled = focusNext(isLine, DIRECTION.BACKWARD);
+      break;
     
     case kCharCodeForEnter:
       handled = handleEnterKeyPress();