From eb95518ef9b1bf9160f0e95d45811a4ef8c0e1fb Mon Sep 17 00:00:00 2001 From: Thomas Bruederli Date: Thu, 1 May 2014 10:54:45 +0200 Subject: [PATCH] Enable keyboard navigation on treelist widgets with cursor keys --- program/js/treelist.js | 93 ++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 89 insertions(+), 4 deletions(-) diff --git a/program/js/treelist.js b/program/js/treelist.js index 0dbedd256..fa3907894 100644 --- a/program/js/treelist.js +++ b/program/js/treelist.js @@ -52,6 +52,7 @@ function rcube_treelist_widget(node, p) indexbyid = {}, selection = null, drag_active = false, + has_focus = false, box_coords = {}, item_coords = [], autoexpand_timer, @@ -105,8 +106,19 @@ function rcube_treelist_widget(node, p) } }); + container.on('focusin', function(e){ + // TODO: only accept focus on virtual nodes from keyboard events + has_focus = true; + }) + .on('focusout', function(e){ + has_focus = false; + }); + container.attr('role', 'tree'); + $(document.body) + .bind('keydown', keypress); + /////// private methods @@ -156,13 +168,13 @@ function rcube_treelist_widget(node, p) function select(id) { if (selection) { - id2dom(selection).removeClass('selected'); + id2dom(selection).removeClass('selected').removeAttr('aria-selected'); selection = null; } var li = id2dom(id); if (li.length) { - li.addClass('selected'); + li.addClass('selected').attr('aria-selected', 'true'); selection = id; // TODO: expand all parent nodes if collapsed scroll_to_node(li); @@ -368,6 +380,7 @@ function rcube_treelist_widget(node, p) var li = $('
  • ') .attr('id', p.id_prefix + (p.id_encode ? p.id_encode(node.id) : node.id)) + .attr('role', 'treeitem') .addClass((node.classes || []).join(' ')); if (replace) @@ -388,7 +401,7 @@ function rcube_treelist_widget(node, p) // add child list and toggle icon if (node.children && node.children.length) { $('
     
    ').appendTo(li); - var ul = $('