From: Michael M Slusarz <slusarz@curecanti.org>
Date: Mon, 20 Apr 2009 19:47:20 +0000 (-0600)
Subject: Bug #8186: Fix keyboard nav in IMP mailbox view
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=cad1f9c3cb4150add6458922209c7f328b651431;p=horde.git

Bug #8186: Fix keyboard nav in IMP mailbox view
---

diff --git a/imp/js/src/mailbox.js b/imp/js/src/mailbox.js
index 5f77f3834..01eea4efa 100644
--- a/imp/js/src/mailbox.js
+++ b/imp/js/src/mailbox.js
@@ -16,13 +16,7 @@ var ImpMailbox = {
 
     selectRow: function(id, select)
     {
-        if (select) {
-            id.addClassName('selectedRow');
-        } else {
-            // Make sure to remove both regular and -over versions.
-            id.removeClassName('selectedRow').removeClassName('selectedRow-over');
-        }
-
+        [ id ].invoke(select ? 'addClassName' : 'removeClassName', 'selectedRow');
         id.down('INPUT.checkbox').setValue(select);
     },
 
@@ -272,7 +266,7 @@ var ImpMailbox = {
     {
         var elt = e.element(),
             key = e.keyCode,
-            loc, search;
+            loc, search, tmp;
 
         if (e.altKey || e.ctrlKey) {
             if (!(key == Event.KEY_UP || key == Event.KEY_DOWN)) {
@@ -304,16 +298,15 @@ var ImpMailbox = {
                     break;
                 }
             } else {
-                search = Event.KEY_DOWN ? 'first' : 'last';
+                search = key == Event.KEY_DOWN ? 'first' : 'last';
             }
 
             if (search) {
-                this.cursor = (search == 'first')
-                    ? $('messages').select('TABLE.messageList TR[id]').first()
-                    : $('messages').select('TABLE.messageList TR[id]').last();
+                tmp = $('messages').select('TABLE.messageList TR[id]');
+                this.cursor = (search == 'first') ? tmp.first() : tmp.last();
             }
 
-            this.cursor.down('TD a.mboxSubject').focus();
+            this.cursor.down('TD A.mboxSubject').focus();
         } else if (key == 32 && this.cursor) {
             this.selectRow(this.cursor, !$F(this.cursor.down('INPUT.checkbox')));
         } else if (!e.shiftKey) {