summaryrefslogtreecommitdiff
path: root/plugins/kolab_files/lib/kolab_files_engine.php
diff options
context:
space:
mode:
authorAleksander Machniak <machniak@kolabsys.com>2014-06-12 10:55:34 (GMT)
committerAleksander Machniak <machniak@kolabsys.com>2014-06-12 10:55:34 (GMT)
commitd1e3acf1ade7a66fa6a0eb42d83bd8229f91186d (patch)
tree3fa65455d7647efdb6c66366b0e6237932387934 /plugins/kolab_files/lib/kolab_files_engine.php
parent6d1ffc345da80e9ac28d26de751ebf394d68b959 (diff)
downloadroundcubemail-plugins-kolab-d1e3acf1ade7a66fa6a0eb42d83bd8229f91186d.tar.gz
Files: Accessibility improvements (#3087), Display dialogs from frames in main window.
Diffstat (limited to 'plugins/kolab_files/lib/kolab_files_engine.php')
-rw-r--r--plugins/kolab_files/lib/kolab_files_engine.php25
1 files changed, 16 insertions, 9 deletions
diff --git a/plugins/kolab_files/lib/kolab_files_engine.php b/plugins/kolab_files/lib/kolab_files_engine.php
index e48ccf8..978bef5 100644
--- a/plugins/kolab_files/lib/kolab_files_engine.php
+++ b/plugins/kolab_files/lib/kolab_files_engine.php
@@ -298,7 +298,7 @@ class kolab_files_engine
$thead = '';
foreach ($this->file_list_head($attrib, $a_show_cols) as $cell) {
- $thead .= html::tag('td', array('class' => $cell['className'], 'id' => $cell['id']), $cell['html']);
+ $thead .= html::tag('th', array('class' => $cell['className'], 'id' => $cell['id']), $cell['html']);
}
return html::tag('table', $attrib,
@@ -332,14 +332,21 @@ class kolab_files_engine
$a_sort_cols = $this->sort_cols;
if (!empty($attrib['optionsmenuicon'])) {
- $onclick = 'return ' . JS_OBJECT_NAME . ".command('menu-open', 'filelistmenu')";
- if ($attrib['optionsmenuicon'] === true || $attrib['optionsmenuicon'] == 'true')
- $list_menu = html::div(array('onclick' => $onclick, 'class' => 'listmenu',
- 'id' => 'listmenulink', 'title' => $this->rc->gettext('listoptions')));
- else
- $list_menu = html::a(array('href' => '#', 'onclick' => $onclick),
- html::img(array('src' => $skin_path . $attrib['optionsmenuicon'],
- 'id' => 'listmenulink', 'title' => $this->rc->gettext('listoptions'))));
+ $onclick = 'return ' . JS_OBJECT_NAME . ".command('menu-open', 'filelistmenu', this, event)";
+ $inner = $this->rc->gettext('listoptions');
+
+ if (is_string($attrib['optionsmenuicon']) && $attrib['optionsmenuicon'] != 'true') {
+ $inner = html::img(array('src' => $skin_path . $attrib['optionsmenuicon'], 'alt' => $RCMAIL->gettext('listoptions')));
+ }
+
+ $list_menu = html::a(array(
+ 'href' => '#list-options',
+ 'onclick' => $onclick,
+ 'class' => 'listmenu',
+ 'id' => 'listmenulink',
+ 'title' => $this->rc->gettext('listoptions'),
+ 'tabindex' => '0',
+ ), $inner);
}
else {
$list_menu = '';