From 9d4104f44a86c96f1b10267e01a69c5653200737 Mon Sep 17 00:00:00 2001 From: 0m364 <13804150+0m364@users.noreply.github.com> Date: Sun, 28 Jun 2026 10:53:00 +0000 Subject: [PATCH] Add proper roles and keyboard support to anchor buttons Added `role="button"` and `aria-haspopup="dialog"` to all OSINT anchor tags that act as modal buttons to improve screen reader semantic meaning. Also added a `keydown` listener for the Spacebar in `modal.js` to ensure the link triggers like a real button for keyboard users. --- .Jules/palette.md | 4 ++++ concepts.html | 2 +- contact.html | 2 +- modal.js | 6 ++++++ products.html | 2 +- 5 files changed, 13 insertions(+), 3 deletions(-) diff --git a/.Jules/palette.md b/.Jules/palette.md index 7c7c523..90ea8bc 100644 --- a/.Jules/palette.md +++ b/.Jules/palette.md @@ -29,3 +29,7 @@ ## 2026-06-12 - Robust Skip-to-Content Links **Learning:** Adding a "Skip to main content" link is a critical accessibility requirement for keyboard users. To make it robust, it needs to be hidden visually but appear when focused using `transform: translateY(-100%)` instead of magic pixel numbers, and the target container (e.g., `
`) must have a matching `id` and `tabindex="-1"` so it can programmatically receive focus across all browsers. **Action:** Always include a visually hidden 'Skip to main content' link immediately after the opening `` tag. Set the main content container with an `id` and explicitly `tabindex="-1"`. + +## 2026-06-28 - Anchor Links as Buttons Accessibility +**Learning:** Using `` tags (with `href="#"`) to trigger interactive elements like modals creates accessibility issues. Screen readers announce them as "links" instead of "buttons", causing confusion about expected behavior. Furthermore, native anchors only trigger on the `Enter` key, whereas keyboard users expect actual buttons to trigger on both `Enter` and `Spacebar`. Pressing `Spacebar` on an anchor without a specific listener causes the page to unexpectedly scroll down. +**Action:** When using `` tags as buttons, enforce semantic meaning by adding `role="button"` and `aria-haspopup="dialog"` (if opening a modal). Additionally, explicitly bind a `keydown` event listener for the `Spacebar` key that calls `e.preventDefault()` to stop page scrolling before executing the button's action. diff --git a/concepts.html b/concepts.html index d4e1807..13b8873 100644 --- a/concepts.html +++ b/concepts.html @@ -19,7 +19,7 @@

CHARLES WILLIAMS

Products Concepts Contact - OSINT + OSINT
diff --git a/contact.html b/contact.html index b30b503..15d119b 100644 --- a/contact.html +++ b/contact.html @@ -49,7 +49,7 @@

Get in Touch

- + OSINT diff --git a/modal.js b/modal.js index 994d6bd..3fd335d 100644 --- a/modal.js +++ b/modal.js @@ -56,6 +56,12 @@ const openModal = (e) => { const btns = document.querySelectorAll('#sitrep-btn'); btns.forEach(btn => { btn.addEventListener('click', openModal); + btn.addEventListener('keydown', (e) => { + if (e.key === ' ' || e.code === 'Space') { + e.preventDefault(); + openModal(e); + } + }); }); // Close modal via button diff --git a/products.html b/products.html index f5f0835..01f1a6c 100644 --- a/products.html +++ b/products.html @@ -20,7 +20,7 @@

CHARLES WILLIAMS

Products Concepts Contact - OSINT + OSINT