From 4bf0c7a4ef13800414735711a46eec180f5dcfdc Mon Sep 17 00:00:00 2001
From: Frederic Guillot <contact@fredericguillot.com>
Date: Tue, 2 Jul 2013 21:50:53 -0400
Subject: [PATCH] Add vim shortcuts to the help page

---
 assets/js/app.js     | 22 ++++++++--------------
 templates/config.php |  4 ++--
 2 files changed, 10 insertions(+), 16 deletions(-)

diff --git a/assets/js/app.js b/assets/js/app.js
index ae769a9..96ac8d0 100644
--- a/assets/js/app.js
+++ b/assets/js/app.js
@@ -376,32 +376,26 @@
     document.onkeypress = function(e) {
 
         switch (e.keyCode || e.which) {
-            case 112:
+            case 112: // p
+            case 107: // k
                 open_previous_item();
                 break;
-            case 110:
+            case 110: // n
+            case 106: // j
                 open_next_item();
                 break;
-            case 118:
+            case 118: // v
                 open_original_item();
                 break;
-            case 111:
+            case 111: // o
                 open_item();
                 break;
-            case 109:
+            case 109: // m
                 change_item_status();
                 break;
-            case 102:
+            case 102: // f
                 bookmark_item();
                 break;
-
-            //vim like navigation
-            case 107:
-                open_previous_item();
-                break;
-            case 106:
-                open_next_item();
-                break;      
         }
     };
 
diff --git a/templates/config.php b/templates/config.php
index 0fccd65..de0c485 100644
--- a/templates/config.php
+++ b/templates/config.php
@@ -42,8 +42,8 @@
     <div class="alert alert-normal">
         <h3><?= t('Keyboard shortcuts') ?></h3>
         <ul>
-            <li><?= t('Previous item') ?> = <strong>p</strong></li>
-            <li><?= t('Next item') ?> = <strong>n</strong></li>
+            <li><?= t('Previous item') ?> = <strong>p</strong> <?= t('or') ?> <strong>j</strong></li>
+            <li><?= t('Next item') ?> = <strong>n</strong> <?= t('or') ?> <strong>k</strong></li>
             <li><?= t('Mark as read or unread') ?> = <strong>m</strong></li>
             <li><?= t('Open original link') ?> = <strong>v</strong></li>
             <li><?= t('Open item') ?> = <strong>o</strong></li>