[PR #6632] [MERGED] feat: files media improvements #7499

New issue

Closed

opened 2026-03-23 23:19:54 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:19:54 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6632
Author: @Xazin
Created: 10/28/2024
Status: ✅ Merged
Merged: 11/4/2024
Merged by: @Xazin

Base: main ← Head: feat/files-media-improvements

📝 Commits (10+)

0f014f9 feat: files & media uiux improvements
312a851 chore: merge branch 'upstream/main' into feat/files-media-improvements
b69405e feat: desktop grid media cell changes
4086193 test: amend tests
a3215fc feat: media mobile improvements
5cb5238 feat: more refinement
d59cdfa chore: merge branch 'upstream/main' into feat/files-media-improvements
a57b987 fix: dart linter
2920940 chore: update collab
7547380 chore: merge branch 'upstream/main' into feat/files-media-improvements

📊 Changes

47 files changed (+1877 additions, -1220 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/desktop/database/database_media_test.dart (+15 -29)
📝 frontend/appflowy_flutter/integration_test/desktop/database/database_row_cover_test.dart (+1 -3)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_file_test.dart (+1 -3)
📝 frontend/appflowy_flutter/integration_test/shared/common_operations.dart (+12 -0)
➕ frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_media_upload.dart (+96 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/option_text_field.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/field/mobile_field_bottom_sheets.dart (+4 -4)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/field/mobile_full_field_editor.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/media_cell_bloc.dart (+12 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/cell/desktop_grid/desktop_grid_media_cell.dart (+111 -64)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/cell/desktop_row_detail/desktop_row_detail_media_cell.dart (+515 -440)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/cell_editor/media_cell_editor.dart (+383 -279)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/cell_editor/mobile_media_cell_editor.dart (+110 -210)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/field/field_type_list.dart (+3 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/field/type_option_editor/media.dart (+6 -9)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/media_file_type_ext.dart (+8 -6)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_detail.dart (+60 -51)
📝 frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_document.dart (+2 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_drop_handler.dart (+29 -19)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_upload_menu.dart (+91 -60)

...and 27 more files

📄 Description

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/6632 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 10/28/2024 **Status:** ✅ Merged **Merged:** 11/4/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `feat/files-media-improvements` --- ### 📝 Commits (10+) - [`0f014f9`](https://github.com/AppFlowy-IO/AppFlowy/commit/0f014f9bef329e516bfa9f13c8913bb845707a91) feat: files & media uiux improvements - [`312a851`](https://github.com/AppFlowy-IO/AppFlowy/commit/312a8517661a0d6aac2025353ca9a877361d552d) chore: merge branch 'upstream/main' into feat/files-media-improvements - [`b69405e`](https://github.com/AppFlowy-IO/AppFlowy/commit/b69405edbf8f047f70e462d8d3832fd658c82d51) feat: desktop grid media cell changes - [`4086193`](https://github.com/AppFlowy-IO/AppFlowy/commit/4086193c6735f441396a1652b99375952af72e03) test: amend tests - [`a3215fc`](https://github.com/AppFlowy-IO/AppFlowy/commit/a3215fcf27eaaf850eeb0c765f89a90546a9dbdb) feat: media mobile improvements - [`5cb5238`](https://github.com/AppFlowy-IO/AppFlowy/commit/5cb5238177b2614d0c971e394368796921b9c598) feat: more refinement - [`d59cdfa`](https://github.com/AppFlowy-IO/AppFlowy/commit/d59cdfadd4c4e02437b28e45efb4444f3f58625e) chore: merge branch 'upstream/main' into feat/files-media-improvements - [`a57b987`](https://github.com/AppFlowy-IO/AppFlowy/commit/a57b987236539d9389624aeaa79ca6e95aa4b124) fix: dart linter - [`2920940`](https://github.com/AppFlowy-IO/AppFlowy/commit/2920940b88647d1c596fa8d3023fd7b3ff68b2f2) chore: update collab - [`7547380`](https://github.com/AppFlowy-IO/AppFlowy/commit/75473807df8a8e15d3cf50dbaee421c1f511cd63) chore: merge branch 'upstream/main' into feat/files-media-improvements ### 📊 Changes **47 files changed** (+1877 additions, -1220 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_media_test.dart` (+15 -29) 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_row_cover_test.dart` (+1 -3) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_file_test.dart` (+1 -3) 📝 `frontend/appflowy_flutter/integration_test/shared/common_operations.dart` (+12 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_media_upload.dart` (+96 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/option_text_field.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/field/mobile_field_bottom_sheets.dart` (+4 -4) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/field/mobile_full_field_editor.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/media_cell_bloc.dart` (+12 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/cell/desktop_grid/desktop_grid_media_cell.dart` (+111 -64) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/cell/desktop_row_detail/desktop_row_detail_media_cell.dart` (+515 -440) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/cell_editor/media_cell_editor.dart` (+383 -279) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/cell_editor/mobile_media_cell_editor.dart` (+110 -210) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/field/field_type_list.dart` (+3 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/field/type_option_editor/media.dart` (+6 -9) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/media_file_type_ext.dart` (+8 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_detail.dart` (+60 -51) 📝 `frontend/appflowy_flutter/lib/plugins/database/widgets/row/row_document.dart` (+2 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_drop_handler.dart` (+29 -19) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/file/file_upload_menu.dart` (+91 -60) _...and 27 more files_ </details> ### 📄 Description #### 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. - [x] 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>