Skip to content

Fix: Restore focus indicator on secondary link (WCAG 2.4.7)#56

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/restore-focus-indicator-secondary-link
Draft

Fix: Restore focus indicator on secondary link (WCAG 2.4.7)#56
Copilot wants to merge 2 commits intomainfrom
copilot/restore-focus-indicator-secondary-link

Conversation

Copy link
Copy Markdown

Copilot AI commented Nov 25, 2025

The .secondary-link:focus rule was setting outline: none, removing the visible focus indicator and violating WCAG 2.1 Level A Success Criterion 2.4.7 (Focus Visible).

Changes

  • Removed the .secondary-link:focus { outline: none; } rule from style.css
  • Secondary link now inherits the global focus style: 2px solid var(--primary-color) with 2px offset

Screenshot

Focus indicator visible on secondary link

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • googlechromelabs.github.io
    • Triggering command: /usr/local/bin/node node install.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>🔴 CRITICAL: Restore focus indicator on secondary link</issue_title>
<issue_description>## Problem
The secondary link has outline: none on focus, completely removing the focus indicator.

Impact

  • Keyboard users cannot see which element has focus
  • Violates WCAG 2.1 Level A Success Criterion 2.4.7 (Focus Visible)
  • Makes keyboard navigation impossible for many users
  • Critical accessibility failure

File to Update

  • style.css - Line 143 (.secondary-link:focus)

Current Code (VIOLATION)

.secondary-link:focus {
  outline: none;  /* ❌ REMOVES focus indicator - WCAG violation */
}

Recommended Solution

Simply delete this rule to use the global focus style:

*:focus {
  outline: 2px solid var(--primary-color);
  outline-offset: 2px;
}

Acceptance Criteria

  • Remove or replace outline: none declaration
  • Focus indicator visible when tabbing to secondary link
  • Focus style consistent with other interactive elements
  • Tested with keyboard navigation only

Testing Instructions

  1. Use keyboard (Tab key) to navigate through the page
  2. Verify visible focus indicator appears on secondary link
  3. Test with different browsers

References

Priority

🔴 HIGH - Critical accessibility violation</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

…or (WCAG 2.4.7)

Co-authored-by: danielmeppiel <51440732+danielmeppiel@users.noreply.github.com>
Copilot AI changed the title [WIP] Restore focus indicator for secondary link Fix: Restore focus indicator on secondary link (WCAG 2.4.7) Nov 25, 2025
Copilot AI requested a review from danielmeppiel November 25, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🔴 CRITICAL: Restore focus indicator on secondary link

2 participants