Skip to content

Conversation

@JammingBen
Copy link
Contributor

As per title.

@JammingBen JammingBen self-assigned this Dec 3, 2025
Copilot AI review requested due to automatic review settings December 3, 2025 14:53
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 adds support for previewing MOV video files by adding the video/quicktime MIME type and implementing a workaround for browser compatibility. The key change addresses the fact that modern browsers expect the video/mp4 MIME type for QuickTime videos.

Key Changes:

  • Added video/quicktime MIME type to the supported media types list
  • Implemented MIME type conversion logic to map video/quicktime to video/mp4 for browser compatibility

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
packages/web-app-preview/src/mimeTypes.ts Adds video/quicktime to the list of supported MIME types
packages/web-app-preview/src/components/Sources/MediaVideo.vue Implements computed property to convert QuickTime MIME type to MP4 for browser compatibility

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

@JammingBen JammingBen force-pushed the feat/preview-add-mov-support branch from a7a93bb to 1293dce Compare December 3, 2025 14:55
@kulmann kulmann merged commit caa1190 into main Dec 4, 2025
28 checks passed
@kulmann kulmann deleted the feat/preview-add-mov-support branch December 4, 2025 06:19
openclouders pushed a commit that referenced this pull request Dec 4, 2025
feat(preview): add support for mov video files
@openclouders openclouders mentioned this pull request Dec 4, 2025
1 task
@openclouders openclouders mentioned this pull request Dec 15, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants