[PR #5199] [MERGED] feat: video block support #6698

New issue

Closed

opened 2026-03-23 23:16:15 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:16:15 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/5199
Author: @Xazin
Created: 4/24/2024
Status: ✅ Merged
Merged: 5/30/2024
Merged by: @Xazin

Base: main ← Head: feat/video-block-support

📝 Commits (10+)

c83da88 feat: video block support
4496248 chore: workaround for ci failing
8aa3ed3 chore: test
e34dfe2 chore: check status
ab541b4 chore: revert apt-get
f8e3e59 chore: add mpv
849a2a8 chore: add mpv to appimagebuilder
6a4c277 chore: try again
af1cfb5 chore: merge branch 'upstream/main' into feat/video-block-support
a5821f5 chore: update after merge

📊 Changes

20 files changed (+890 additions, -51 deletions)

View changed files

📝 .github/workflows/flutter_ci.yaml (+20 -2)
📝 frontend/appflowy_flutter/android/app/src/main/AndroidManifest.xml (+7 -0)
📝 frontend/appflowy_flutter/ios/Podfile.lock (+36 -0)
📝 frontend/appflowy_flutter/ios/Runner/Info.plist (+5 -0)
📝 frontend/appflowy_flutter/lib/main.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart (+13 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/custom_image_block_component.dart (+3 -5)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/add_block_toolbar_item.dart (+11 -0)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/upload_video_menu.dart (+93 -0)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/video_menu.dart (+314 -0)
➕ frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/video_placeholder.dart (+136 -0)
📝 frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart (+9 -0)
📝 frontend/appflowy_flutter/pubspec.lock (+170 -2)
📝 frontend/appflowy_flutter/pubspec.yaml (+1 -1)
➕ frontend/resources/flowy_icons/16x/m_add_block_video.svg (+2 -0)
📝 frontend/resources/translations/en.json (+10 -1)
📝 frontend/scripts/docker-buildfiles/Dockerfile (+1 -1)
📝 frontend/scripts/install_dev_env/install_linux.sh (+11 -1)
📝 frontend/scripts/linux_distribution/appimage/AppImageBuilder.yml (+44 -36)

📄 Description

Tested on Linux (Ubuntu), MacOS, Windows, Android & iOS

Feature Preview

Mobile

https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/22a8bb05-361d-48cb-becd-dc497d9427d4

https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/f970b87b-9472-4da0-a567-0bdcdd13e928

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/5199 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 4/24/2024 **Status:** ✅ Merged **Merged:** 5/30/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `feat/video-block-support` --- ### 📝 Commits (10+) - [`c83da88`](https://github.com/AppFlowy-IO/AppFlowy/commit/c83da881bd7211a4c608a430652a1289326d7ac1) feat: video block support - [`4496248`](https://github.com/AppFlowy-IO/AppFlowy/commit/4496248c324191b2babc4ef2a19c8a452f8bfc0d) chore: workaround for ci failing - [`8aa3ed3`](https://github.com/AppFlowy-IO/AppFlowy/commit/8aa3ed3eb81c5ef4587ac8b118eefb2099de1d2f) chore: test - [`e34dfe2`](https://github.com/AppFlowy-IO/AppFlowy/commit/e34dfe2273a3176a499543cfef94ba301e80c063) chore: check status - [`ab541b4`](https://github.com/AppFlowy-IO/AppFlowy/commit/ab541b40d82de6210ce11c10342623b53becad9e) chore: revert apt-get - [`f8e3e59`](https://github.com/AppFlowy-IO/AppFlowy/commit/f8e3e59e8d032d00f6c4ed778b9aa73ee53278d4) chore: add mpv - [`849a2a8`](https://github.com/AppFlowy-IO/AppFlowy/commit/849a2a8816c87ff4085a6db4b1dea74e3f8ab997) chore: add mpv to appimagebuilder - [`6a4c277`](https://github.com/AppFlowy-IO/AppFlowy/commit/6a4c277082bb02164a5586fad1b7019bdf0ae4a2) chore: try again - [`af1cfb5`](https://github.com/AppFlowy-IO/AppFlowy/commit/af1cfb548fd06edca1b9ad1efd62ea8e83430072) chore: merge branch 'upstream/main' into feat/video-block-support - [`a5821f5`](https://github.com/AppFlowy-IO/AppFlowy/commit/a5821f5c778f8cf176648d419b10c09f8ea6e939) chore: update after merge ### 📊 Changes **20 files changed** (+890 additions, -51 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/flutter_ci.yaml` (+20 -2) 📝 `frontend/appflowy_flutter/android/app/src/main/AndroidManifest.xml` (+7 -0) 📝 `frontend/appflowy_flutter/ios/Podfile.lock` (+36 -0) 📝 `frontend/appflowy_flutter/ios/Runner/Info.plist` (+5 -0) 📝 `frontend/appflowy_flutter/lib/main.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart` (+13 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/image/custom_image_block_component.dart` (+3 -5) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/mobile_toolbar_v3/add_block_toolbar_item.dart` (+11 -0) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/upload_video_menu.dart` (+93 -0) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/video_menu.dart` (+314 -0) ➕ `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/video/video_placeholder.dart` (+136 -0) 📝 `frontend/appflowy_flutter/lib/shared/patterns/common_patterns.dart` (+9 -0) 📝 `frontend/appflowy_flutter/pubspec.lock` (+170 -2) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+1 -1) ➕ `frontend/resources/flowy_icons/16x/m_add_block_video.svg` (+2 -0) 📝 `frontend/resources/translations/en.json` (+10 -1) 📝 `frontend/scripts/docker-buildfiles/Dockerfile` (+1 -1) 📝 `frontend/scripts/install_dev_env/install_linux.sh` (+11 -1) 📝 `frontend/scripts/linux_distribution/appimage/AppImageBuilder.yml` (+44 -36) </details> ### 📄 Description Tested on Linux (Ubuntu), MacOS, Windows, Android & iOS ### Feature Preview <details><summary>Mobile</summary> https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/22a8bb05-361d-48cb-becd-dc497d9427d4 </details> https://github.com/AppFlowy-IO/AppFlowy/assets/42929161/f970b87b-9472-4da0-a567-0bdcdd13e928 #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>