Skip to content

Commit

Permalink
WICKET-7131 Improved accessibility and screen reader support (#1034)
Browse files Browse the repository at this point in the history
* Improved accessability and screen reader support

* Ensure that the input field exists before trying to change its attributes

* Fix code style issues

---------

Co-authored-by: Erik Strid <[email protected]>
(cherry picked from commit b25c831)
  • Loading branch information
strido authored and martin-g committed Nov 19, 2024
1 parent e8d583a commit 219d45c
Showing 1 changed file with 16 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@
showAutoComplete();
}
render(true, false);
jqEvent.preventDefault();
}

break;
Expand All @@ -153,6 +154,7 @@
render(true, false);
showAutoComplete();
}
jqEvent.preventDefault();
}

break;
Expand Down Expand Up @@ -243,7 +245,7 @@
{
// Remove the autocompletion menu if still present from
// a previous call. This is required to properly register
// the mouse event handler again
// the mouse event handler again
var choiceDiv=document.getElementById(getMenuId());
if (choiceDiv !== null) {
choiceDiv.parentNode.parentNode.removeChild(choiceDiv.parentNode);
Expand Down Expand Up @@ -325,7 +327,6 @@
container.appendChild(choiceDiv);
choiceDiv.id=getMenuId();
choiceDiv.className = "wicket-aa";
choiceDiv.ariaLive = "polite";
}


Expand Down Expand Up @@ -456,7 +457,10 @@
hideAutoCompleteTimer = undefined;

var input = Wicket.$(ajaxAttributes.c);
$(input).attr("aria-expanded", "false");
if (input) {
input.setAttribute("aria-expanded", "false");
input.removeAttribute("aria-activedescendant");
}

visible = 0;
setSelected(-1);
Expand Down Expand Up @@ -628,6 +632,7 @@
selChSinceLastRender = true; // selected item will not have selected style until rendrered
}
element.innerHTML=resp;
element.firstChild.role = "listbox";
var selectableElements = getSelectableElements();
if (selectableElements) {
elementCount=selectableElements.length;
Expand Down Expand Up @@ -675,6 +680,11 @@
node.onclick = clickFunc;
node.onmouseover = mouseOverFunc;
node.onmousedown = mouseDownFunc;
node.role = "option";
node.id = getMenuId() + '-item-' + i;
node.setAttribute("tabindex", -1);
node.setAttribute("aria-posinset", i + 1);
node.setAttribute("aria-setsize", elementCount);
}
} else {
elementCount=0;
Expand Down Expand Up @@ -770,6 +780,8 @@
var node=getSelectableElement(0);
var re = /\bselected\b/gi;
var sizeAffected = false;
var input=Wicket.$(ajaxAttributes.c);

for(var i=0;i<elementCount;i++)
{
var origClassNames = node.className;
Expand All @@ -780,6 +792,7 @@

if (node && node instanceof HTMLElement && node.attributes) {
node.setAttribute("aria-selected", "true");
input.setAttribute("aria-activedescendant", node.id);
}

if (adjustScroll) {
Expand Down

0 comments on commit 219d45c

Please sign in to comment.