Skip to content

feat: add progress steps indicator (Upload → Configure → Export → Download) - #1669

Open
arnavgoel17 wants to merge 1 commit into
reframe-oss:mainfrom
arnavgoel17:feat/progress-steps-indicator
Open

arnavgoel17 wants to merge 1 commit into
reframe-oss:mainfrom
arnavgoel17:feat/progress-steps-indicator

Conversation

@arnavgoel17

Copy link
Copy Markdown

Summary

  • Adds a 4-step workflow stepper at the top of the editor: Upload → Configure → Export → Download.
  • Step state derives from the editor's file/status (per issue mapping): Upload when no file, Configure when a file is loaded and idle, Export during loading-engine/exporting, Download when done.
  • Completed steps show a checkmark; the active step is highlighted with the accent color; connectors fill as steps complete.
  • Accessible: wrapped in a labelled nav + ol, each step uses aria-current="step" when active, and a visually-hidden state label is provided for screen readers. Respects prefers-reduced-motion.

Acceptance Criteria

  • 4-step indicator visible at top of page
  • Active step highlighted
  • Completed steps show checkmark
  • Accessible with ARIA

Test plan

  • npx tsc --noEmit — passes
  • npm run lint — passes
  • npm run build — builds successfully

Fixes #98

🤖 Generated with Claude Code

…nload)

Adds a 4-step workflow stepper at the top of the editor that maps to the
video editor status, with active/completed states, checkmarks for completed
steps, and ARIA labelling for accessibility (Fixes reframe-oss#98).

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 16, 2026

Copy link
Copy Markdown

@arnavgoel17 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

👋 Thanks for your PR, @arnavgoel17!

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

Copy link
Copy Markdown
Contributor

✅ PR Format Check Passed — @arnavgoel17

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 github-actions Bot added level:intermediate Intermediate level - 35 pts type:bug Bug fix type:testing Testing labels Jul 16, 2026
@github-actions

github-actions Bot commented Aug 7, 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 Aug 7, 2026
@magic-peach magic-peach added the status:awaiting-review Clean PR waiting on a maintainer, not the author label Aug 14, 2026
@github-actions github-actions Bot removed the needs-fixes PR has issues that need to be fixed label Aug 15, 2026
@github-actions

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 Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:intermediate Intermediate level - 35 pts needs-fixes PR has issues that need to be fixed status:awaiting-review Clean PR waiting on a maintainer, not the author type:bug Bug fix type:testing Testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Enhancement] Add progress steps indicator (Upload → Configure → Export → Download)

2 participants