Skip to content

Feature/upload requirements info#1602

Open
Harshwardhan-1 wants to merge 4 commits into
magic-peach:mainfrom
Harshwardhan-1:feature/upload-requirements-info
Open

Feature/upload requirements info#1602
Harshwardhan-1 wants to merge 4 commits into
magic-peach:mainfrom
Harshwardhan-1:feature/upload-requirements-info

Conversation

@Harshwardhan-1

Copy link
Copy Markdown
Contributor

Description

This PR adds a clear "Upload Requirements" information section in the video upload component. It helps users understand file limitations before uploading, reducing upload/export failures and confusion.

The section includes:

  • Maximum file size limit
  • Warning threshold for large files
  • Supported video formats
  • Recommended codec (H.264 MP4)
  • Browser processing limitation (FFmpeg.wasm memory constraints)

Related Issue

Closes #1601

Type of Contribution

  • New feature

Participant Info

  • GitHub username: YOUR_GITHUB_USERNAME
  • Contribution level (Beginner/Intermediate/Advanced): Beginner

Screen Recording

Demonstrates file upload flow with requirement section visible before upload, including drag & drop and file selection behavior.

Recording / Loom link: https://loom.com/your-video-link

Checklist

  • I have read the contribution guidelines
  • My changes follow the project structure
  • I have tested my changes in Chrome, Firefox, and Safari
  • bun run lint passes (no ESLint errors)
  • bunx tsc --noEmit passes (no TypeScript errors)
  • New interactive elements have aria-label / accessible names
  • No console.log statements left in
  • This PR is related to a valid issue
  • Screen recording attached above (required for UI/feature/design changes)

@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

@Harshwardhan-1 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 Issues — @Harshwardhan-1

Please fix the following before your PR can be reviewed:

  • ⚠️ Use a conventional PR title. Examples:
    • feat: add dark mode support
    • fix: resolve aria label missing on slider
    • docs: add deployment guide to README

Push new commits after fixing — this comment will update automatically.

📖 CONTRIBUTING.md

@github-actions github-actions Bot added level:advanced Advanced level - 55 pts type:design UI/UX design type:feature New feature labels Jun 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @Harshwardhan-1!

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! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:advanced Advanced level - 55 pts type:design UI/UX design type:feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Display upload requirements and supported file limitations before video upload

1 participant