Skip to content

Conversation

@arbirali
Copy link

@arbirali arbirali commented May 6, 2025

Description

This pull request resolves several accessibility issues in the Tutor Indigo theme that were identified during a recent accessibility review. These updates aim to improve usability for all users, especially those relying on assistive technologies. The changes also help ensure compliance with recognized accessibility standards, such as the Web Content Accessibility Guidelines (WCAG) 2.1.

For further details, please refer to the related issue: overhangio/tutor-indigo#146.


Accessibility Fixes Implemented

Add Accessible Titles to Image-Based Links

This pull request improves accessibility by adding visible titles to image-based links throughout the Tutor Indigo theme. Previously, these links lacked text descriptions, making them difficult to interpret for users relying on screen readers or keyboard navigation.

With this update, descriptive title attributes are added to each image-based link, providing additional context and enhancing the overall usability of the interface for all users.


Accessibility Audit Report

For the full audit and issue breakdown, please refer to the following report: Accessibility Audit

@wittjeff
Copy link

The missing visible text is not a WCAG conformance issue. Adding title attributes to things that have accessible names may add additional verbal redundancy for screen reader users (depending on implementation). Not urgent for or against.

@arbirali
Copy link
Author

arbirali commented Jun 5, 2025

@wittjeff
Thanks for the feedback! You're right about the potential for redundancy with screen readers. In this case, I added the title attribute to the logo since it doesn’t contain any visible text, and the title helps provide additional context for sighted users on hover. Since it doesn’t conflict with an existing accessible name, I saw it as a small usability improvement. Open to suggestions if there’s a better approach!

@DawoudSheraz
Copy link

Please update the PR base branch to indigo/sumac or indigo/teak. indigo branch is not an active branch.

@arbirali arbirali changed the base branch from indigo to teak/indigo June 10, 2025 10:53
@arbirali arbirali closed this Jun 10, 2025
@arbirali
Copy link
Author

arbirali commented Jun 10, 2025

The issue https://tree.taiga.io/project/zaraahmed-tutor-indigo-accessibility/us/79 has already been addressed in the #13 pull request. So there is no need for this PR.

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.

4 participants