Skip to content

fix(a11y): announce FFmpeg loading and export status via aria-live region - #1586

Closed
sweetylearner-max wants to merge 1 commit into
magic-peach:mainfrom
sweetylearner-max:fix/ffmpeg-loading-aria-announcement
Closed

fix(a11y): announce FFmpeg loading and export status via aria-live region#1586
sweetylearner-max wants to merge 1 commit into
magic-peach:mainfrom
sweetylearner-max:fix/ffmpeg-loading-aria-announcement

Conversation

@sweetylearner-max

Copy link
Copy Markdown

Related Issue

Closes #168

Description of Changes

  • Added announcement state and prevStatusRef to track status transitions in ExportOverlay.tsx
  • Added a <div role="status" aria-live="polite" className="sr-only"> region that announces:
    • "Loading video processing engine, please wait..." when the engine starts loading
    • "Export started. Processing your video locally." when export begins
  • Announcement clears when the overlay is dismissed

Acceptance Criteria

  • 'Loading engine' announced to screen reader
  • 'Export started' announced when complete
  • Uses polite live region (doesn't interrupt)

Type of Change

  • Accessibility improvement

@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

@sweetylearner-max is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown
Contributor

✅ PR Format Check Passed — @sweetylearner-max

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@github-actions

Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @sweetylearner-max!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

👋 This PR has been inactive for 21 days. It will be automatically closed in 14 days if there's no further activity.

If you need help with this PR, please comment or ping a maintainer.
Make sure your branch is rebased onto main and all CI checks pass.

@github-actions github-actions Bot added the needs-fixes PR has issues that need to be fixed label Jul 9, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 Closing this PR due to inactivity. Feel free to reopen and rebase onto main if you'd like to continue!

@github-actions github-actions Bot closed this Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:beginner Beginner level - 20 pts needs-fixes PR has issues that need to be fixed type:bug Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Accessibility] Add 'Loading' state announcement when FFmpeg engine loads

1 participant