From bc2accc455e1dab67e0d29f034deddd9401ecad2 Mon Sep 17 00:00:00 2001 From: alecpl Date: Thu, 25 Mar 2010 18:16:50 +0000 Subject: [PATCH] - Added Home/End kayboard keys support on lists (#1486430) --- CHANGELOG | 1 + program/js/app.js | 15 ++------- program/js/list.js | 77 ++++++++++++++++++++++++++++++++++++++++------ 3 files changed, 70 insertions(+), 23 deletions(-) diff --git a/CHANGELOG b/CHANGELOG index f7905dd84..fa24842b3 100644 --- a/CHANGELOG +++ b/CHANGELOG @@ -1,6 +1,7 @@ CHANGELOG RoundCube Webmail =========================== +- Added Home/End kayboard keys support on lists (#1486430) - Added possibility to select all messages in a folder (#1484756) - Added 'imap_force_caps' option for after-login CAPABILITY checking (#1485750) - Password: Support dovecotpw encryption diff --git a/program/js/app.js b/program/js/app.js index 0fc5e7675..5e5bfe06f 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -1927,7 +1927,7 @@ function rcube_webmail() if (!this.env.threading) return; - var root = this.find_thread_root(uid); + var root = this.message_list.find_root(uid); if (uid == root) return; @@ -1947,17 +1947,6 @@ function rcube_webmail() this.set_unread_children(root); }; - // finds root message for specified thread - this.find_thread_root = function(uid) - { - var r = this.message_list.rows[uid]; - - if (r.parent_uid) - return this.find_thread_root(r.parent_uid); - else - return uid; - } - // update thread indicators for all messages in a thread below the specified message // return number of removed/added root level messages this.update_thread = function (uid) @@ -1975,7 +1964,7 @@ function rcube_webmail() count--; else if (row.unread) { // update unread_children for thread root - var parent = this.find_thread_root(uid); + var parent = this.message_list.find_root(uid); rows[parent].unread_children--; this.set_unread_children(parent); } diff --git a/program/js/list.js b/program/js/list.js index 614d9c3c7..13fa7d321 100644 --- a/program/js/list.js +++ b/program/js/list.js @@ -320,6 +320,20 @@ click_row: function(e, id) }, +/* + * Returns thread root ID for specified row ID + */ +find_root: function(uid) +{ + var r = this.rows[uid]; + + if (r && r.parent_uid) + return this.find_root(r.parent_uid); + else + return uid; +}, + + expand_row: function(e, id) { var row = this.rows[id]; @@ -379,6 +393,7 @@ expand: function(row) row.expanded = true; depth = row.depth; new_row = row.obj.nextSibling; + this.update_expando(row.uid, true); this.triggerEvent('expandcollapse', { uid:row.uid, expanded:row.expanded }); } else { @@ -427,6 +442,7 @@ collapse_all: function(row) row.expanded = false; depth = row.depth; new_row = row.obj.nextSibling; + this.update_expando(row.uid); this.triggerEvent('expandcollapse', { uid:row.uid, expanded:row.expanded }); // don't collapse sub-root tree in multiexpand mode @@ -450,9 +466,7 @@ collapse_all: function(row) $(new_row).hide(); if (r.has_children) { r.expanded = false; - var expando = document.getElementById('rcmexpando' + r.uid); - if (expando) - expando.className = 'collapsed'; + this.update_expando(r.uid); this.triggerEvent('expandcollapse', { uid:r.uid, expanded:r.expanded }); } } @@ -472,6 +486,7 @@ expand_all: function(row) row.expanded = true; depth = row.depth; new_row = row.obj.nextSibling; + this.update_expando(row.uid, true); this.triggerEvent('expandcollapse', { uid:row.uid, expanded:row.expanded }); } else { @@ -490,9 +505,7 @@ expand_all: function(row) $(new_row).show(); if (r.has_children) { r.expanded = true; - var expando = document.getElementById('rcmexpando' + r.uid); - if (expando) - expando.className = 'expanded'; + this.update_expando(r.uid, true); this.triggerEvent('expandcollapse', { uid:r.uid, expanded:r.expanded }); } } @@ -502,6 +515,14 @@ expand_all: function(row) return false; }, +update_expando: function(uid, expanded) +{ + var expando = document.getElementById('rcmexpando' + uid); + if (expando) + expando.className = expanded ? 'expanded' : 'collapsed'; +}, + + /** * get first/next/previous/last rows that are not hidden */ @@ -648,16 +669,39 @@ select_next: function() this.select_row(new_row.uid, false, false); }, + /** * Select first row */ -select_first: function() +select_first: function(mod_key) { - var first_row = this.get_first_row(); - if (first_row) - this.select_row(first_row, false, false); + var row = this.get_first_row(); + if (row && mod_key) { + this.shift_select(row, mod_key); + this.triggerEvent('select'); + this.scrollto(row); + } + else if (row) + this.select(row); }, + +/** + * Select last row + */ +select_last: function(mod_key) +{ + var row = this.get_last_row(); + if (row && mod_key) { + this.shift_select(row, mod_key); + this.triggerEvent('select'); + this.scrollto(row); + } + else if (row) + this.select(row); +}, + + /** * Add all childs of the given row to selection */ @@ -903,6 +947,12 @@ key_press: function(e) this.key_pressed = keyCode; this.triggerEvent('keypress'); return ret; + case 36: // Home + this.select_first(mod_key); + return rcube_event.cancel(e); + case 35: // End + this.select_last(mod_key); + return rcube_event.cancel(e); default: this.shiftkey = e.shiftKey; this.key_pressed = keyCode; @@ -1007,6 +1057,13 @@ scrollto: function(id) { var scroll_to = Number(row.offsetTop); + // expand thread if target row is hidden (collapsed) + if (!scroll_to && this.rows[id].parent_uid) { + var parent = this.find_root(this.rows[id].uid); + this.expand_all(this.rows[parent]); + scroll_to = Number(row.offsetTop); + } + if (scroll_to < Number(this.frame.scrollTop)) this.frame.scrollTop = scroll_to; else if (scroll_to + Number(row.offsetHeight) > Number(this.frame.scrollTop) + Number(this.frame.offsetHeight))