From 9240c96339b26989f060c57c61a2677e2f1e502e Mon Sep 17 00:00:00 2001 From: Thomas Bruederli <thomas@roundcube.net> Date: Wed, 18 Jun 2014 08:35:48 -0400 Subject: [PATCH] Improve accessibility on attachments list: use custom tabindex attribute + add aria-label for meaningful voice output on delete icons --- program/js/app.js | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/program/js/app.js b/program/js/app.js index 3fd1823..b4a2250 100644 --- a/program/js/app.js +++ b/program/js/app.js @@ -4070,7 +4070,7 @@ if (!att.complete && att.frame) att.html = '<a title="'+this.get_label('cancel')+'" onclick="return rcmail.cancel_attachment_upload(\''+name+'\', \''+att.frame+'\');" href="#cancelupload" class="cancelupload">' - + (this.env.cancelicon ? '<img src="'+this.env.cancelicon+'" alt="" />' : this.get_label('cancel')) + '</a>' + att.html; + + (this.env.cancelicon ? '<img src="'+this.env.cancelicon+'" alt="'+this.get_label('cancel')+'" />' : this.get_label('cancel')) + '</a>' + att.html; var indicator, li = $('<li>'); @@ -4087,6 +4087,10 @@ li.appendTo(this.gui_objects.attachmentlist); } + // set tabindex attribute + var tabindex = $(this.gui_objects.attachmentlist).attr('data-tabindex') || '0'; + li.find('a').attr('tabindex', tabindex); + if (upload_id && this.env.attachments[upload_id]) delete this.env.attachments[upload_id]; -- Gitblit v1.9.1