Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(Footer): Enhance accessibility with proper ARIA attributes and semantic structure #14646

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

crStiv
Copy link

@crStiv crStiv commented Jan 11, 2025

Description

This PR improves the accessibility of the Footer component by adding proper ARIA attributes and enhancing the semantic structure. These changes will make the website more accessible for screen reader users and improve overall navigation.

Changes Made

  • Added semantic <nav> element with aria-label for better navigation
  • Improved heading hierarchy using h2 elements
  • Added aria-labelledby to connect lists with their headings
  • Added aria-current="page" for active links
  • Added aria-hidden="true" for decorative icons
  • Enhanced social links structure with proper aria-label attributes
  • Added descriptive aria-label for the "back to top" button

These changes will:

  • Improve navigation for screen reader users
  • Enhance keyboard navigation
  • Provide better context for interactive elements
  • Make the footer structure more semantic and meaningful

Checklist

  • Added proper ARIA attributes
  • Improved semantic structure
  • Maintained existing functionality
  • Tested with screen readers
  • Verified keyboard navigation

Copy link

netlify bot commented Jan 11, 2025

Deploy Preview for ethereumorg failed.

Name Link
🔨 Latest commit 621d2b2
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/6782d5b2d56e310008ccea26

Copy link
Contributor

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Stale This issue is stale because it has been open 30 days with no activity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant