From 3bf96c3eb4e806b36892b6b0c23e892bfcfe33cd Mon Sep 17 00:00:00 2001
From: milanmajchrak <milan.majchrak@dataquest.sk>
Date: Thu, 10 Sep 2026 19:26:01 +0200
Subject: [PATCH] Port #1084 (part) to dtq-dev-9-base: accessible labels and
 per-component ids for the search boxes

Card X-02a, part B of three. Seven templates that the v9 upgrade took wholesale from vanilla 9.3, so
the accessibility hunks of 5a92fdfa89 were never applied. Every one of these inputs is either
unnamed for a screen reader or named in hardcoded English regardless of the UI language.

What changes:

* a visually-hidden <label> is added for each control - two in eperson-search-box (the scope select
  and the query input), three in starts-with-date (year, month, free-text date), one each in
  group-search-box, members-list, subgroups-list, groups-registry and search-navbar;
* aria-label="Search input" and aria-label="Search scope", which were hardcoded English strings even
  in the Czech UI, become bindings on the keys that already exist unreferenced in en.json5:
  labelPrefix + 'search.input', labelPrefix + 'search.scope' and messagePrefix + '.search.input'.
  No i18n key is added, removed or renamed;
* the hardcoded id="query" is replaced by a per-component id in all five components that carry it in
  this set. formControlName and name stay "query"/"scope"; only the DOM id changes.

Three fork mistakes are ported fixed rather than literally, each for a reason that is checkable on
this branch:

1. class="sr-only" does not exist in Bootstrap 5 and 9-base defines it nowhere
   (git grep -nE '^[[:space:]]*\.sr-only' -- src => 0 hits, "bootstrap": "^5.3"). A literal port would
   render a visible label in the middle of every search form. The class used is visually-hidden,
   which is defined at src/styles/_global-styles.scss:280 and is position:absolute, so it adds
   nothing to the input-group layout.
2. <label for="startsWith + 'input'"> and <label for="'query' + '-input'"> are static attributes, not
   bindings: they emit those characters literally and pair with nothing. The ids here are static, so
   the labels use plain static for= values that match the real id.
3. The group edit page renders ds-members-list and ds-subgroups-list side by side
   (group-form.component.html:60 and :65) and both hardcoded id="query". Adding <label for="query">
   without disambiguating first would have bound both labels to the first input.

The fork also gave starts-with-date [id]="startsWith + 'input'". startsWith is mutated as the user
types (starts-with-date.component.ts:123), so the id would change on every keystroke and the label
association would break. A static id="startsWith-input" is used instead.

Tests: 5a92fdfa89 ships none. eperson-search-box.component.spec.ts gains cases asserting that each
control's label resolves to that control (label.htmlFor === input.id, id non-empty) and that the
accessible name is the translated key rather than the literal "Search input". Proven load-bearing
with negative controls - see the PR description.

Source: 5a92fdfa89 (dtq-dev PR #1084), partial - the suggestion inputs from the same commit are in
the sibling PR for card X-02a part C.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
---
 .../members-list/members-list.component.html  |  5 ++--
 .../members-list.component.spec.ts            | 13 ++++++++
 .../subgroups-list.component.html             |  5 ++--
 .../subgroups-list.component.spec.ts          | 13 ++++++++
 .../groups-registry.component.html            |  3 +-
 .../search-navbar.component.html              |  3 +-
 .../eperson-search-box.component.html         |  9 ++++--
 .../eperson-search-box.component.spec.ts      | 30 +++++++++++++++++++
 .../group-search-box.component.html           |  5 ++--
 .../date/starts-with-date.component.html      |  5 +++-
 10 files changed, 79 insertions(+), 12 deletions(-)

diff --git a/src/app/access-control/group-registry/group-form/members-list/members-list.component.html b/src/app/access-control/group-registry/group-form/members-list/members-list.component.html
index 072b836d236..e1ccd869678 100644
--- a/src/app/access-control/group-registry/group-form/members-list/members-list.component.html
+++ b/src/app/access-control/group-registry/group-form/members-list/members-list.component.html
@@ -84,8 +84,9 @@ <h3 id="search" class="border-bottom pb-2">
   <form [formGroup]="searchForm" (ngSubmit)="search(searchForm.value)" class="d-flex justify-content-between">
     <div class="flex-grow-1 me-3">
       <div class="mb-3 input-group me-3">
-        <input type="text" name="query" id="query" formControlName="query"
-          class="form-control" aria-label="Search input">
+        <label for="members-list-query" class="visually-hidden">{{messagePrefix + '.search.input' | translate}}</label>
+        <input type="text" name="query" id="members-list-query" formControlName="query"
+          class="form-control" [attr.aria-label]="messagePrefix + '.search.input' | translate">
         <span class="input-group-append">
           <button type="submit" class="search-button btn btn-primary">
             <i class="fas fa-search"></i> {{ messagePrefix + '.search.button' | translate }}</button>
diff --git a/src/app/access-control/group-registry/group-form/members-list/members-list.component.spec.ts b/src/app/access-control/group-registry/group-form/members-list/members-list.component.spec.ts
index 0271dd4aad4..c5edf6f5093 100644
--- a/src/app/access-control/group-registry/group-form/members-list/members-list.component.spec.ts
+++ b/src/app/access-control/group-registry/group-form/members-list/members-list.component.spec.ts
@@ -199,6 +199,19 @@ describe('MembersListComponent', () => {
     expect(comp).toBeDefined();
   }));
 
+  it('should label its search input with an id of its own, not the generic one', () => {
+    const input: HTMLInputElement = fixture.nativeElement.querySelector('input[name="query"]');
+    const label: HTMLLabelElement = fixture.nativeElement.querySelector('label[for="' + input.id + '"]');
+
+    expect(input.id).toBeTruthy();
+    // ds-members-list and ds-subgroups-list render side by side on the group edit page,
+    // so a shared generic id makes one label point at the other component's input
+    expect(input.id).not.toEqual('query');
+    expect(label).toBeTruthy();
+    expect(label.htmlFor).toEqual(input.id);
+    expect(Array.from(label.classList)).toContain('visually-hidden');
+  });
+
   describe('current members list', () => {
     it('should show list of eperson members of current active group', () => {
       const epersonIdsFound = fixture.debugElement.queryAll(By.css('#ePeopleMembersOfGroup tr td:first-child'));
diff --git a/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.html b/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.html
index d09ab08c42a..36818d8b32e 100644
--- a/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.html
+++ b/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.html
@@ -69,8 +69,9 @@ <h4 id="search" class="border-bottom pb-2">
   <form [formGroup]="searchForm" (ngSubmit)="search(searchForm.value)" class="d-flex justify-content-between">
     <div class="flex-grow-1 me-3">
       <div class="mb-3 input-group me-3">
-        <input type="text" name="query" id="query" formControlName="query"
-          class="form-control" aria-label="Search input">
+        <label for="subgroups-list-query" class="visually-hidden">{{messagePrefix + '.search.input' | translate}}</label>
+        <input type="text" name="query" id="subgroups-list-query" formControlName="query"
+          class="form-control" [attr.aria-label]="messagePrefix + '.search.input' | translate">
         <span class="input-group-append">
           <button type="submit" class="search-button btn btn-primary">
             <i class="fas fa-search"></i> {{ messagePrefix + '.search.button' | translate }}
diff --git a/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.spec.ts b/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.spec.ts
index 819db3d1c0d..a0d3e302013 100644
--- a/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.spec.ts
+++ b/src/app/access-control/group-registry/group-form/subgroup-list/subgroups-list.component.spec.ts
@@ -217,6 +217,19 @@ describe('SubgroupsListComponent', () => {
     expect(comp).toBeDefined();
   }));
 
+  it('should label its search input with an id of its own, not the generic one', () => {
+    const input: HTMLInputElement = fixture.nativeElement.querySelector('input[name="query"]');
+    const label: HTMLLabelElement = fixture.nativeElement.querySelector('label[for="' + input.id + '"]');
+
+    expect(input.id).toBeTruthy();
+    // ds-members-list and ds-subgroups-list render side by side on the group edit page,
+    // so a shared generic id makes one label point at the other component's input
+    expect(input.id).not.toEqual('query');
+    expect(label).toBeTruthy();
+    expect(label.htmlFor).toEqual(input.id);
+    expect(Array.from(label.classList)).toContain('visually-hidden');
+  });
+
   describe('current subgroup list', () => {
     it('should show list of subgroups of current active group', () => {
       const groupIdsFound = fixture.debugElement.queryAll(By.css('#subgroupsOfGroup tr td:first-child'));
diff --git a/src/app/access-control/group-registry/groups-registry.component.html b/src/app/access-control/group-registry/groups-registry.component.html
index 2c6c2590e2f..4bb759fe66c 100644
--- a/src/app/access-control/group-registry/groups-registry.component.html
+++ b/src/app/access-control/group-registry/groups-registry.component.html
@@ -16,7 +16,8 @@ <h2 id="search" class="border-bottom pb-2">{{messagePrefix + 'search.head' | tra
       <form [formGroup]="searchForm" (ngSubmit)="search(searchForm.value)" class="d-flex justify-content-between">
         <div class="flex-grow-1 me-3">
           <div class="mb-3 input-group">
-            <input type="text" name="query" id="query" formControlName="query"
+            <label for="groups-registry-query" class="visually-hidden">{{messagePrefix + 'search.placeholder' | translate}}</label>
+            <input type="text" name="query" id="groups-registry-query" formControlName="query"
               class="form-control" [attr.aria-label]="messagePrefix + 'search.placeholder' | translate"
               [placeholder]="(messagePrefix + 'search.placeholder' | translate)" >
             <span class="input-group-append">
diff --git a/src/app/search-navbar/search-navbar.component.html b/src/app/search-navbar/search-navbar.component.html
index ba7430bd181..2e9ba6e9f2d 100644
--- a/src/app/search-navbar/search-navbar.component.html
+++ b/src/app/search-navbar/search-navbar.component.html
@@ -1,7 +1,8 @@
 <div [title]="'nav.search' | translate" (dsClickOutside)="collapse()">
   <div class="d-inline-block position-relative">
     <form [formGroup]="searchForm" (ngSubmit)="onSubmit(searchForm.value)" autocomplete="on" class="d-flex">
-      <input #searchInput [@toggleAnimation]="isExpanded" [attr.aria-label]="('nav.search' | translate)" name="query"
+      <label for="search-navbar-query" class="visually-hidden">{{ 'nav.search' | translate }}</label>
+      <input #searchInput id="search-navbar-query" [@toggleAnimation]="isExpanded" [attr.aria-label]="('nav.search' | translate)" name="query"
              formControlName="query" type="text" placeholder="{{searchExpanded ? ('nav.search' | translate) : ''}}"
              class="d-inline-block bg-transparent position-absolute form-control dropdown-menu-end p1"
              [class.display]="searchExpanded ? 'inline-block' : 'none'"
diff --git a/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.html b/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.html
index 214f90cf279..793e5fa820c 100644
--- a/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.html
+++ b/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.html
@@ -2,15 +2,18 @@
       [formGroup]="searchForm"
       (ngSubmit)="submit(searchForm.value); $event.stopImmediatePropagation();" >
   <div>
-    <select name="scope" id="scope" formControlName="scope" class="form-select" aria-label="Search scope">
+    <label for="eperson-search-scope" class="visually-hidden">{{labelPrefix + 'search.scope' | translate}}</label>
+    <select name="scope" id="eperson-search-scope" formControlName="scope" class="form-select"
+            [attr.aria-label]="labelPrefix + 'search.scope' | translate">
       <option value="metadata">{{labelPrefix + 'search.scope.metadata' | translate}}</option>
       <option value="email">{{labelPrefix + 'search.scope.email' | translate}}</option>
     </select>
   </div>
   <div class="flex-grow-1 me-3 ms-3">
     <div class="mb-3 input-group">
-      <input type="text" name="query" id="query" formControlName="query"
-             class="form-control" aria-label="Search input">
+      <label for="eperson-search-query" class="visually-hidden">{{labelPrefix + 'search.input' | translate}}</label>
+      <input type="text" name="query" id="eperson-search-query" formControlName="query"
+             class="form-control" [attr.aria-label]="labelPrefix + 'search.input' | translate">
     <button type="submit" class="search-button btn btn-primary">
       <i class="fas fa-search"></i> {{ labelPrefix + 'search.button' | translate }}
     </button>
diff --git a/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.spec.ts b/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.spec.ts
index 63fe58478af..2a1cf73df91 100644
--- a/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.spec.ts
+++ b/src/app/shared/eperson-group-list/eperson-search-box/eperson-search-box.component.spec.ts
@@ -95,6 +95,36 @@ describe('EpersonSearchBoxComponent test suite', () => {
       expect(comp.searchForm.controls.query.value).toBe('');
     });
 
+    it('should label the scope select and the query input, each resolving to its own control', () => {
+      fixture.detectChanges();
+      const element: HTMLElement = fixture.nativeElement;
+      const labels: HTMLLabelElement[] = Array.from(element.querySelectorAll('label'));
+
+      expect(labels.length).toEqual(2);
+      labels.forEach((label: HTMLLabelElement) => {
+        expect(label.htmlFor).toBeTruthy();
+        const control: HTMLElement = element.querySelector('[id="' + label.htmlFor + '"]');
+        expect(control).toBeTruthy();
+        expect(control.id).toEqual(label.htmlFor);
+        // the label must not be rendered visibly (Bootstrap 5 dropped the old screen-reader class)
+        expect(Array.from(label.classList)).toContain('visually-hidden');
+      });
+    });
+
+    it('should name both controls from the translated keys rather than hardcoded English', () => {
+      fixture.detectChanges();
+      const input: HTMLInputElement = fixture.nativeElement.querySelector('input[name="query"]');
+      const select: HTMLSelectElement = fixture.nativeElement.querySelector('select[name="scope"]');
+
+      // TranslateModule.forRoot() with no catalogue echoes the key back, so the key is the observable value
+      expect(input.getAttribute('aria-label')).toEqual('admin.access-control.epeople.search.input');
+      expect(select.getAttribute('aria-label')).toEqual('admin.access-control.epeople.search.scope');
+      // and the id is no longer the generic one that other components on the same page also used
+      expect(input.id).toBeTruthy();
+      expect(input.id).not.toEqual('query');
+      expect(select.id).not.toEqual('scope');
+    });
+
     it('should emit new search event', () => {
       const data = {
         scope: 'metadata',
diff --git a/src/app/shared/eperson-group-list/group-search-box/group-search-box.component.html b/src/app/shared/eperson-group-list/group-search-box/group-search-box.component.html
index efb01625911..0614533602d 100644
--- a/src/app/shared/eperson-group-list/group-search-box/group-search-box.component.html
+++ b/src/app/shared/eperson-group-list/group-search-box/group-search-box.component.html
@@ -3,8 +3,9 @@
       (ngSubmit)="submit(searchForm.value); $event.stopImmediatePropagation();" >
   <div class="flex-grow-1 me-3">
     <div class="mb-3 input-group">
-      <input type="text" name="query" id="query" formControlName="query"
-             class="form-control" aria-label="Search input">
+      <label for="group-search-query" class="visually-hidden">{{labelPrefix + 'search.input' | translate}}</label>
+      <input type="text" name="query" id="group-search-query" formControlName="query"
+             class="form-control" [attr.aria-label]="labelPrefix + 'search.input' | translate">
         <button type="submit" class="search-button btn btn-primary">
           <i class="fas fa-search"></i> {{ labelPrefix + 'search.button' | translate }}
         </button>
diff --git a/src/app/shared/starts-with/date/starts-with-date.component.html b/src/app/shared/starts-with/date/starts-with-date.component.html
index 7c80d357a8a..83453dffd2f 100644
--- a/src/app/shared/starts-with/date/starts-with-date.component.html
+++ b/src/app/shared/starts-with/date/starts-with-date.component.html
@@ -4,6 +4,7 @@
       {{ 'browse.startsWith.jump' | translate }}
     </span>
     <div class="col-6 col-md-2">
+      <label for="year-select" class="visually-hidden">{{ 'browse.startsWith.choose_year.label' | translate }}</label>
       <select id="year-select" class="form-select" (change)="setStartsWithYearEvent($event)" [attr.aria-label]="'browse.startsWith.choose_year.label' |translate">
         <option [value]="-1" [selected]="!startsWithYear">
           {{ 'browse.startsWith.choose_year' | translate }}
@@ -18,6 +19,7 @@
       </select>
     </div>
     <div class="col-6 col-md-2">
+      <label for="month-select" class="visually-hidden">{{ 'browse.startsWith.months.none.label' | translate }}</label>
       <select id="month-select" class="form-select" (change)="setStartsWithMonthEvent($event)" [attr.aria-label]="'browse.startsWith.months.none.label' |translate">
         @for (option of monthOptions; track option) {
           <option
@@ -30,7 +32,8 @@
     </div>
     <div class="col-12 col-md-6">
       <div class="mb-3 input-group pt-1 pt-md-0">
-        <input class="form-control" placeholder="{{'browse.startsWith.type_date' | translate}}" [attr.aria-label]="'browse.startsWith.type_date.label' |translate" type="text" name="startsWith"
+        <label for="startsWith-input" class="visually-hidden">{{ 'browse.startsWith.type_date.label' | translate }}</label>
+        <input id="startsWith-input" class="form-control" placeholder="{{'browse.startsWith.type_date' | translate}}" [attr.aria-label]="'browse.startsWith.type_date.label' |translate" type="text" name="startsWith"
           formControlName="startsWith" [value]="getStartsWith() ? getStartsWith() : ''"/>
         <button class="btn btn-primary" type="submit" role="button" tabindex="0"><i class="fas fa-book-open"></i> {{ 'browse.startsWith.submit' | translate }}</button>
       </div>