Keyboard shortcuts for 'Accept/Revert url/test' buttons

CHANGELOG.md

@@ -32,6 +32,7 @@ All notable changes to AET will be documented in this file.
 - [PR-338](https://github.com/Cognifide/aet/pull/338) Enable creating new dbs by default
 - [PR-345](https://github.com/Cognifide/aet/pull/345) Deprecate AET maven plugin
 - [PR-301](https://github.com/Cognifide/aet/pull/301) New versions of virtualization tools used with new AET cookbook
+- [PR-360](https://github.com/Cognifide/aet/pull/360) Keyboard shortcuts for 'Accept/Revert url/test' buttons ([#317](https://github.com/Cognifide/aet/issues/317))
 
 ## Version 2.1.6
 

documentation/src/main/wiki/SuiteReportFeatures.md

@@ -62,4 +62,7 @@ Blue version means that it is a rebased version of previous suite - this is also
 * press **e** to expand/collapse all error items
 * use **[** / **]** to navigate between test's urls
 * press **m** to show/hide layout mask (when available)
-* press **←** / **→** to navigate between url's tabs
+* press **←** / **→** to navigate between url's tabs
+* press **a** for accepting suite/test/url
+* press **r** for reverting suite/test/url
+* **shift** modifier that will define that the specific opened test case should be accepted/reverted (shift + A - accepts opened test case ; shift + R - reverts opened test case )

documentation/src/main/wiki/releases/2.0.0/Documentation-2.0.0.md

@@ -3410,6 +3410,9 @@ It is possible to search tests and URLs by the query. Searched fields are the UR
 * use **[** / **]** to navigate between test's urls
 * press **m** to show/hide layout mask (when available)
 * press **←** / **→** to navigate between url's tabs
+* press **a** for accepting suite/test/url
+* press **r** for reverting suite/test/url
+* **shift** modifier that will define that the specific opened test case should be accepted/reverted (shift + A - accepts opened test case ; shift + R - reverts opened test case )
 
 # How It Works
 AET is distributed system that consists of modules deployed on 

documentation/src/main/wiki/releases/2.1.0/Documentation-2.1.0.md

@@ -3565,6 +3565,9 @@ It is possible to search tests and URLs by the query. Searched fields are the UR
 * use **[** / **]** to navigate between test's urls
 * press **m** to show/hide layout mask (when available)
 * press **←** / **→** to navigate between url's tabs
+* press **a** for accepting suite/test/url
+* press **r** for reverting suite/test/url
+* **shift** modifier that will define that the specific opened test case should be accepted/reverted (shift + A - accepts opened test case ; shift + R - reverts opened test case )
 
 # How It Works
 AET is a distributed system that consists of a set of modules deployed on 

documentation/src/main/wiki/releases/2.1.3/Documentation-2.1.3.md

@@ -4061,6 +4061,9 @@ It is possible to search tests and URLs by the query. Searched fields are the UR
 * use **[** / **]** to navigate between test's urls
 * press **m** to show/hide layout mask (when available)
 * press **←** / **→** to navigate between url's tabs
+* press **a** for accepting suite/test/url
+* press **r** for reverting suite/test/url
+* **shift** modifier that will define that the specific opened test case should be accepted/reverted (shift + A - accepts opened test case ; shift + R - reverts opened test case )
 
 # How It Works
 AET is a distributed system that consists of a set of modules deployed on 

report/src/main/webapp/app/components/keyboardShortcuts.directive.js

@@ -37,6 +37,8 @@ define(['angularAMD', 'userSettingsService'], function (angularAMD) {
        code 81 - q
        code 219 - [
        code 221 - ]
+       code 65 - a
+       code 82 - r
        */
 
       var shortcuts = {
@@ -60,14 +62,16 @@ define(['angularAMD', 'userSettingsService'], function (angularAMD) {
         },
         '221': function () {
           scope.traverseTree('down');
-        }
+        },
+        '65': accept,
+        '82': revert
       };
 
       $(document).on('keydown', function (e) {
-        if ($('body').hasClass('modal-open')) {
+        if ($('body').hasClass('modal-open') || $(e.target).hasClass('search-input')) {
           return true;
         } else if (shortcuts[e.keyCode]) {
-          shortcuts[e.keyCode]();
+          shortcuts[e.keyCode](e);
         }
       });
 
@@ -86,6 +90,20 @@ define(['angularAMD', 'userSettingsService'], function (angularAMD) {
         }
       };
 
+      function clickButton(buttonClass, event) {
+        var $tab = (event.shiftKey) ? $('.tab-content-toolbar') : $('.toolbar-bottom');
+        var $btns = $tab.find('.toolbar-btns');
+        $btns.find(buttonClass).click();
+      }
+
+      function accept(event) {
+        clickButton('.button-blue', event);
+      }
+
+      function revert(event) {
+        clickButton('.button-red', event);
+      }
+
       function goToTab(direction) {
         var $tabs = $('.test-tabs'),
             $active = $tabs.find('.active');
@@ -251,7 +269,7 @@ define(['angularAMD', 'userSettingsService'], function (angularAMD) {
         var $nextTestTabs = $('.nav-tabs').children();
         $nextTestTabs.each(function() {
           if($(this).text().replace(/\s/g, '') === currentTabLabel) {
-            $(this).find('a').click(); 
+            $(this).find('a').click();
             currentTabLabel = null;
           }
         });
@@ -260,15 +278,15 @@ define(['angularAMD', 'userSettingsService'], function (angularAMD) {
       //MutationObserver fires a callback every time something changes on the page
       //and here it's used to click a tab before the page is actually rendered to get rid of flickering effect
       var mutationObserver = new MutationObserver(callback);
-      
+
       function callback(mutList) {
         function findElement(element) {
           if($(element.target).hasClass('nav-tabs')) {
             clickOnTab();
             return true;
           }
         }
-        mutList.find(findElement);     
+        mutList.find(findElement);
       }
 
       mutationObserver.observe(document, {

report/src/main/webapp/report.html

@@ -67,6 +67,9 @@ <h4 class="modal-title">Handful keyboard shortcuts</h4>
             available)
           </li>
           <li>press <span class="label label-info">← / →</span> to navigate between url's tabs</li>
+          <li>press <span class="label label-info">a</span> for accepting suite/test/url</li>
+          <li>press <span class="label label-info">r</span> for reverting suite/test/url</li>
+          <li><span class="label label-info">shift</span> modifier that will define that the specific opened test case should be accepted/reverted (shift + A - accepts opened test case ; shift + R - reverts opened test case)</li>
         </ul>
       </div>
     </div>