[PR #6238] [MERGED] feat: cover in database row #7267

New issue

Closed

opened 2026-03-23 23:18:50 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:18:50 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6238
Author: @Xazin
Created: 9/8/2024
Status: ✅ Merged
Merged: 9/15/2024
Merged by: @Xazin

Base: main ← Head: feat/database-row-cover

📝 Commits (10+)

2dc1220 feat: cover in database
0428be1 chore: merge 'upstream/main' into feat/database-row-cover
f0e3a75 chore: clean debugPrint
11b0ad0 chore: merge branch 'upstream/main' into feat/database-row-cover
7679b90 chore: update collab rev
6b87a18 feat: add cover functionality
007dd61 chore: merge branch 'upstream/main' into feat/database-row-cover
aa2b3ab fix: provider issue for calendarbloc in popover
48d5ec1 feat: mobile row cover
de0fa5b chore: merge branch 'upstream/main' into feat/database-row-cover

📊 Changes

48 files changed (+1368 additions, -380 deletions)

View changed files

➕ frontend/appflowy_flutter/integration_test/desktop/database/database_row_cover_test.dart (+188 -0)
📝 frontend/appflowy_flutter/integration_test/desktop_runner_2.dart (+2 -0)
📝 frontend/appflowy_flutter/integration_test/desktop_runner_3.dart (+3 -0)
📝 frontend/appflowy_flutter/integration_test/shared/database_test_op.dart (+13 -0)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart (+201 -77)
📝 frontend/appflowy_flutter/lib/mobile/presentation/database/card/mobile_card_content.dart (+51 -38)
📝 frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/media_cell_bloc.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/application/row/row_banner_bloc.dart (+40 -8)
📝 frontend/appflowy_flutter/lib/plugins/database/application/row/row_service.dart (+11 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/board/application/board_bloc.dart (+11 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart (+6 -2)
📝 frontend/appflowy_flutter/lib/plugins/database/board/presentation/widgets/board_hidden_groups.dart (+5 -6)
📝 frontend/appflowy_flutter/lib/plugins/database/calendar/application/calendar_bloc.dart (+10 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_event_card.dart (+5 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_event_editor.dart (+1 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_page.dart (+9 -5)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/grid_bloc.dart (+11 -0)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/row/mobile_row_detail_bloc.dart (+29 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/row/row_detail_bloc.dart (+33 -3)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/application/row/row_document_bloc.dart (+1 -1)

...and 28 more files

📄 Description

Relates: #3491 (But not closes, because we still are missing the ability to add a cover to a database page itself)

Feature Preview

Update:

https://github.com/user-attachments/assets/3634988b-de0c-4f15-a26c-ec5c4c647f4b

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/6238 **Author:** [@Xazin](https://github.com/Xazin) **Created:** 9/8/2024 **Status:** ✅ Merged **Merged:** 9/15/2024 **Merged by:** [@Xazin](https://github.com/Xazin) **Base:** `main` ← **Head:** `feat/database-row-cover` --- ### 📝 Commits (10+) - [`2dc1220`](https://github.com/AppFlowy-IO/AppFlowy/commit/2dc12204d6f671afc2d345a710e80a26c4cc41a9) feat: cover in database - [`0428be1`](https://github.com/AppFlowy-IO/AppFlowy/commit/0428be1d278442b2cdae6822e8e4259a28597d81) chore: merge 'upstream/main' into feat/database-row-cover - [`f0e3a75`](https://github.com/AppFlowy-IO/AppFlowy/commit/f0e3a7548222bd65b9c7ce856c7fcd765c3b29be) chore: clean debugPrint - [`11b0ad0`](https://github.com/AppFlowy-IO/AppFlowy/commit/11b0ad0c51e69b8d3a100e576468915a5418153f) chore: merge branch 'upstream/main' into feat/database-row-cover - [`7679b90`](https://github.com/AppFlowy-IO/AppFlowy/commit/7679b90cb1301efeeb7db657ed73be3b24700b9e) chore: update collab rev - [`6b87a18`](https://github.com/AppFlowy-IO/AppFlowy/commit/6b87a18f7c7478e64c89505dcc5e98f143181bba) feat: add cover functionality - [`007dd61`](https://github.com/AppFlowy-IO/AppFlowy/commit/007dd619375a31c8f46a6696648b23d134c2d463) chore: merge branch 'upstream/main' into feat/database-row-cover - [`aa2b3ab`](https://github.com/AppFlowy-IO/AppFlowy/commit/aa2b3ab9825d3bb60257746d589b0792325f15fd) fix: provider issue for calendarbloc in popover - [`48d5ec1`](https://github.com/AppFlowy-IO/AppFlowy/commit/48d5ec14daa8bb19142d840873186ed9d5a4d4e2) feat: mobile row cover - [`de0fa5b`](https://github.com/AppFlowy-IO/AppFlowy/commit/de0fa5b91ecf4d068d287ffe0a13d2808aada7e8) chore: merge branch 'upstream/main' into feat/database-row-cover ### 📊 Changes **48 files changed** (+1368 additions, -380 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/integration_test/desktop/database/database_row_cover_test.dart` (+188 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop_runner_2.dart` (+2 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop_runner_3.dart` (+3 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/database_test_op.dart` (+13 -0) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart` (+201 -77) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/database/card/mobile_card_content.dart` (+51 -38) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/cell/bloc/media_cell_bloc.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/row/row_banner_bloc.dart` (+40 -8) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/row/row_service.dart` (+11 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/application/board_bloc.dart` (+11 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart` (+6 -2) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/presentation/widgets/board_hidden_groups.dart` (+5 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database/calendar/application/calendar_bloc.dart` (+10 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_event_card.dart` (+5 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_event_editor.dart` (+1 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/calendar/presentation/calendar_page.dart` (+9 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/grid_bloc.dart` (+11 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/row/mobile_row_detail_bloc.dart` (+29 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/row/row_detail_bloc.dart` (+33 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/application/row/row_document_bloc.dart` (+1 -1) _...and 28 more files_ </details> ### 📄 Description Relates: #3491 _(But not closes, because we still are missing the ability to add a cover to a database page itself)_ ### Feature Preview **Update**: https://github.com/user-attachments/assets/3634988b-de0c-4f15-a26c-ec5c4c647f4b <img src="https://github.com/user-attachments/assets/1cd97b81-9ba4-4e22-85bc-51c5c557c24f" width="250"/> <img src="https://github.com/user-attachments/assets/f04fb18a-4b52-42b8-9157-3790c37746a1" width="250"/> #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] 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>