Skip to content

feat(flashcards): add alt text support for term and definition images#27

Merged
pganesh-apphelix merged 1 commit intomainfrom
fix/TNL2-518
Feb 27, 2026
Merged

feat(flashcards): add alt text support for term and definition images#27
pganesh-apphelix merged 1 commit intomainfrom
fix/TNL2-518

Conversation

@pganesh-apphelix
Copy link
Contributor

Description

  • Adds accessibility support for Flashcards images by introducing explicit alt text fields for term and definition images.

Jira

Version

  • 1.0.12

Copilot AI review requested due to automatic review settings February 18, 2026 14:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances accessibility for the Flashcards feature by adding explicit alt text fields for term and definition images, allowing content creators to provide more meaningful descriptions than the default fallback to term/definition text.

Changes:

  • Introduced new constants for alt text fields (TERM_IMAGE_ALT, DEFINITION_IMAGE_ALT)
  • Updated backend handlers to process and store alt text data with fallback logic
  • Modified frontend JavaScript to use explicit alt text when rendering images

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
setup.py Bumped version to 1.0.12
games/constants.py Added constants for term and definition image alt text fields
games/handlers/common.py Updated save handler to persist alt text fields
games/handlers/flashcards.py Added logic to extract and provide alt text with fallbacks to term/definition text
games/static/js/src/flashcards.js Updated image rendering to use explicit alt text fields with fallback logic

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pganesh-apphelix pganesh-apphelix merged commit ae1cbe5 into main Feb 27, 2026
8 of 16 checks passed
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.

3 participants