Merge pull request #1409 from danskernesdigitalebibliotek/DDFLSBP-671…

…-add-aria-label-to-delete-row-button DDFLSBP-671: add aria label to delete row button
danskernesdigitalebibliotek · Sep 11, 2024 · 47e8bf9 · 47e8bf9
2 parents 22a3df6 + ec7c552
commit 47e8bf9
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 0 deletions.
diff --git a/src/apps/advanced-search/AdvancedSearch.dev.tsx b/src/apps/advanced-search/AdvancedSearch.dev.tsx
@@ -200,6 +200,11 @@ export default {
       defaultValue: "Add row",
       control: { type: "text" }
     },
+    advancedSearchRemoveRowText: {
+      name: "Remove row",
+      defaultValue: "Remove row @inputNumber",
+      control: { type: "text" }
+    },
     advancedSearchInputPlaceholderText: {
       name: "Advanced search input placeholder",
       defaultValue: "Search term",

diff --git a/src/apps/advanced-search/AdvancedSearch.entry.tsx b/src/apps/advanced-search/AdvancedSearch.entry.tsx
@@ -66,6 +66,7 @@ interface AdvancedSearchEntryTextProps {
   advancedSearchFilterNonFictionText: string;
   advancedSearchFilterHoldingStatusText: string;
   advancedSearchInputLabelText: string;
+  advancedSearchRemoveRowText: string;
 }
 
 interface AdvancedSearchEntryConfigProps {

diff --git a/src/apps/advanced-search/AdvancedSearchRow.tsx b/src/apps/advanced-search/AdvancedSearchRow.tsx
@@ -168,6 +168,9 @@ const AdvancedSearchRow: React.FC<AdvancedSearchRowProps> = ({
             onClick={() => {
               removeRow(rowIndex, setSearchObject);
             }}
+            aria-label={t("advancedSearchRemoveRowText", {
+              placeholders: { "@inputNumber": rowIndex + 1 }
+            })}
           >
             <img className="input-with-dropdown__icon" src={IconMinus} alt="" />
           </button>