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

        [codereviewtool] add more help information
        https://bugs.webkit.org/show_bug.cgi?id=54847

        * code-review.js:

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

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

index 6416d1f..90013c4 100644 (file)
@@ -2,6 +2,15 @@
 
         Reviewed by Adam Barth.
 
+        [codereviewtool] add more help information
+        https://bugs.webkit.org/show_bug.cgi?id=54847
+
+        * code-review.js:
+
+2011-02-20  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
         [codereviewtool] use 'r' to focus the review select element
         https://bugs.webkit.org/show_bug.cgi?id=54845
 
index bbc4a73..134d6f4 100644 (file)
@@ -425,7 +425,15 @@ var CODE_REVIEW_UNITTEST;
     descriptor = comments.length + ' comment';
     if (comments.length > 1)
       descriptor += 's';
-    $('.help').append(' This patch has ' + descriptor + '.  Scroll through them with the "n" and "p" keys.');
+    $('.help .more').before(' This patch has ' + descriptor + '.  Scroll through them with the "n" and "p" keys. ');
+  }
+  
+  function showMoreHelp() {
+    $('.more-help').removeClass('inactive');
+  }
+  
+  function hideMoreHelp() {
+    $('.more-help').addClass('inactive');
   }
 
   function scanForStyleQueueComments(text) {
@@ -1054,6 +1062,21 @@ var CODE_REVIEW_UNITTEST;
     $(document.body).prepend('<div id="message">' +
         '<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>' +
+          '<a href="javascript:" class="more">[more]</a>' +
+          '<div class="more-help inactive">' +
+            '<div class="winter"></div>' +
+            '<div class="lightbox"><table>' +
+              '<tr><td>enter</td><td>add/edit comment for focused item</td></tr>' +
+              '<tr><td>escape</td><td>accept current comment / close preview and help popups</td></tr>' +
+              '<tr><td>j</td><td>focus next diff</td></tr>' +
+              '<tr><td>k</td><td>focus previous diff</td></tr>' +
+              '<tr><td>shift + j</td><td>focus next line</td></tr>' +
+              '<tr><td>shift + k</td><td>focus previous line</td></tr>' +
+              '<tr><td>n</td><td>focus next comment</td></tr>' +
+              '<tr><td>p</td><td>focus previous comment</td></tr>' +
+              '<tr><td>r</td><td>focus review select element</td></tr>' +
+            '</table></div>' +
+          '</div>' +
         '</div>' +
         '</div>');
     $(document.body).append('<div id="toolbar">' +
@@ -1339,6 +1362,8 @@ var CODE_REVIEW_UNITTEST;
   $('.frozenComment').live('click', handleUnfreezeComment);
   $('.comment .discard').live('click', handleDiscardComment);
   $('.comment .ok').live('click', handleAcceptComment);
+  $('.more').live('click', showMoreHelp);
+  $('.more-help .winter').live('click', hideMoreHelp);
 
   function freezeComment(comment_block) {
     var comment_textarea = comment_block.find('textarea');
@@ -1444,6 +1469,11 @@ var CODE_REVIEW_UNITTEST;
     case KEY_CODE.enter:
       handled = handleEnterKeyPress();
       break;
+      
+    case KEY_CODE.escape:
+      hideMoreHelp();
+      handled = true;
+      break;
     }
     
     if (handled)