From 36cccff56efff992d0821b29d54b92f640f9f937 Mon Sep 17 00:00:00 2001
From: Lara van Diemen <lara@yard.nl>
Date: Thu, 11 Dec 2025 14:13:32 +0100
Subject: [PATCH] a11y: remove role navigation from facetwp pager

---
 src/modules/A11yFacetWP.ts | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/modules/A11yFacetWP.ts b/src/modules/A11yFacetWP.ts
index 07c5f9f..2152725 100644
--- a/src/modules/A11yFacetWP.ts
+++ b/src/modules/A11yFacetWP.ts
@@ -59,6 +59,7 @@ export class A11yFacetWP {
 		 * This timeout is necessary because otherwise it will be called too early
 		 */
 		setTimeout( () => {
+			this.removeRoleNavigationFromPager();
 			this.addAriaCurrentToPager();
 			this.changeAriaLabelSelections();
 		}, 1 );
@@ -96,6 +97,18 @@ export class A11yFacetWP {
 		}
 	}
 
+	/**
+	 * A11y: remove 'role=navigation' from FacetWP pager if it is inside a <nav> element.
+	 */
+	private removeRoleNavigationFromPager(): void {
+		const pager = document.querySelector( '.facetwp-pager' );
+		if ( ! pager ) return;
+
+		if ( pager.parentElement?.tagName.toLowerCase() === 'nav' ) {
+			pager.removeAttribute( 'role' );
+		}
+	}
+
 	/**
 	 * A11y: add 'aria-current' attribute to FacetWP pager.
 	 */