2011-02-20 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2011 06:46:05 +0000 (06:46 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2011 06:46:05 +0000 (06:46 +0000)
        Reviewed by Adam Barth.

        [codereviewtool] use 'r' to focus the review select element
        https://bugs.webkit.org/show_bug.cgi?id=54845

        This allows for quick keyboard access to the toolbar items.
        Hit r to get to the review select element. Then you can
        tab to the cq/preview/publish elements.

        * code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79186 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 433cbfe..6416d1f 100644 (file)
@@ -2,6 +2,19 @@
 
         Reviewed by Adam Barth.
 
+        [codereviewtool] use 'r' to focus the review select element
+        https://bugs.webkit.org/show_bug.cgi?id=54845
+
+        This allows for quick keyboard access to the toolbar items.
+        Hit r to get to the review select element. Then you can
+        tab to the cq/preview/publish elements.
+
+        * code-review.js:
+
+2011-02-20  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
         [codereviewtool] don't let lines/diffblocks be mouse focusable
         https://bugs.webkit.org/show_bug.cgi?id=54851
 
index afe0dd5..deab829 100644 (file)
@@ -451,7 +451,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=36"></script>
+<script src="code-review.js?version=37"></script>
 EOF
 
     def self.revisionOrDescription(string)
index c3fb45c..bbc4a73 100644 (file)
@@ -76,7 +76,8 @@ var CODE_REVIEW_UNITTEST;
     j: 74,
     k: 75,
     n: 78,
-    p: 80    
+    p: 80,
+    r: 82
   }
 
   function idForLine(number) {
@@ -1413,6 +1414,11 @@ var CODE_REVIEW_UNITTEST;
     var handled = false;
 
     switch (e.keyCode) {
+    case KEY_CODE.r:
+      $('.review select').focus();
+      handled = true;
+      break;
+
     case KEY_CODE.n:
       handled = focusNext(isComment, DIRECTION.FORWARD);
       break;