From: Michael M Slusarz <slusarz@curecanti.org>
Date: Wed, 23 Dec 2009 09:00:20 +0000 (-0700)
Subject: Better keyboard navigation
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=c2f3c232e47ef4f2e2cf5e6e2b946ee7aa1f2f83;p=horde.git

Better keyboard navigation

Up on first item (or no selection) cancels the selection.
Down on last item doesn't move selection.
---

diff --git a/horde/js/KeyNavList.js b/horde/js/KeyNavList.js
index 026aba658..4a41d1561 100644
--- a/horde/js/KeyNavList.js
+++ b/horde/js/KeyNavList.js
@@ -245,24 +245,18 @@ var KeyNavList = Class.create({
 
     markPrevious: function()
     {
-        var elt;
-
-        if (this.selected) {
-            elt = this.selected.previous();
-        }
-
-        this.markSelected((!this.selected || !elt) ? this.div.down().childElements().last() : elt);
+        this.markSelected(this.selected ? this.selected.previous() : null);
     },
 
     markNext: function()
     {
-        var elt;
+        var elt = this.selected
+            ? this.selected.next()
+            : this.div.down().childElements().first();
 
-        if (this.selected) {
-            elt = this.selected.next();
+        if (elt) {
+            this.markSelected(elt);
         }
-
-        this.markSelected((!this.selected || !elt) ? this.div.down().childElements().first() : elt);
     },
 
     getCurrentEntry: function()