From b63271ca117a9acb4472754c2f0df4016d40a500 Mon Sep 17 00:00:00 2001 From: Lucas Date: Mon, 30 Sep 2024 09:32:39 +0100 Subject: [PATCH] Improve role attributes usage -Replaced role="menu" with role="group" to provide better context for assistive technologies. -Removed role="menuitem" as checkboxes and radio buttons already have appropriate native semantics. --- src/jquery.multi-select.js | 4 +--- src/jquery.multi-select.min.js | 18 +++++++++--------- 2 files changed, 10 insertions(+), 12 deletions(-) diff --git a/src/jquery.multi-select.js b/src/jquery.multi-select.js index 94ba44d..10717cf 100644 --- a/src/jquery.multi-select.js +++ b/src/jquery.multi-select.js @@ -156,7 +156,7 @@ this.$menu = $(this.settings['menuHTML']); this.$menu.attr({ - 'role': 'menu' + 'role': 'group' }).on('keyup.multiselect', function(e){ var key = e.which; var escapeKey = 27; @@ -261,7 +261,6 @@ var $item = $(_this.settings['menuItemHTML']) .attr({ 'for': unique_id, - 'role': 'menuitem' }) .text(' ' + preset.name) .on('keydown.multiselect', _this.upDown.bind(_this, 'preset')) @@ -332,7 +331,6 @@ var $item = $(this.settings['menuItemHTML']) .attr({ 'for': unique_id, - 'role': 'menuitem' }) .on('keydown.multiselect', this.upDown.bind(this, 'menuitem')) .text(' ' + $option.text()); diff --git a/src/jquery.multi-select.min.js b/src/jquery.multi-select.min.js index 20dfc39..d09bb36 100644 --- a/src/jquery.multi-select.min.js +++ b/src/jquery.multi-select.min.js @@ -1,13 +1,13 @@ (function(d){function h(a,b){this.h=d(a);this.g=d.extend({},k,b);this.U()}var k={containerHTML:'
',menuHTML:'
',buttonHTML:'',menuItemsHTML:'
',menuFieldsetHTML:'
',menuFieldsetLegendHTML:'',menuItemHTML:'