[PR #3328] [MERGED] feat: revamp row detail page UI #5679

New issue

Closed

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

mirror commented

2026-03-23 22:19:48 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3328
Author: @richardshiue
Created: 9/4/2023
Status: ✅ Merged
Merged: 9/13/2023
Merged by: @appflowy

Base: main ← Head: row-detail-page-revamp

📝 Commits (10+)

5719e40 feat: revamp row detail page UI
2487044 chore: some minor details and fix tests
ecb1eda fix: fix tests
f9057e5 chore: remove unused field
c980c28 chore: code cleanup
b666a82 test: add reordering fields in row page tests
b90d68c chore: remove duplicate and delete row events
2ac985b chore: merge branch 'main' into row-detail-page-revamp
2672400 chore: timestamp cell ui adjustment
0ccfdeb chore: remove unused code

📊 Changes

23 files changed (+648 additions, -487 deletions)

View changed files

📝 frontend/appflowy_flutter/integration_test/database_calendar_test.dart (+4 -1)
📝 frontend/appflowy_flutter/integration_test/database_row_page_test.dart (+37 -2)
📝 frontend/appflowy_flutter/integration_test/util/database_test_op.dart (+43 -1)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/database_controller.dart (+1 -7)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/row/row_banner_bloc.dart (+1 -5)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/application/row/row_detail_bloc.dart (+31 -19)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_cell.dart (+6 -2)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/row/action.dart (+2 -3)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/accessory/cell_accessory.dart (+22 -57)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cell_builder.dart (+2 -0)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/checkbox_cell/checkbox_cell.dart (+20 -2)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/date_cell/date_cell.dart (+28 -11)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/number_cell/number_cell.dart (+24 -3)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/select_option_cell/extension.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/select_option_cell/select_option_cell.dart (+10 -8)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/text_cell/text_cell.dart (+7 -4)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/timestamp_cell/timestamp_cell.dart (+26 -33)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_action.dart (+30 -113)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_banner.dart (+113 -62)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_detail.dart (+20 -114)

...and 3 more files

📄 Description

Revamp the row detail page. Highlights include:

move duplicate and delete row actions to a sub-menu at the page's top-right corner
allow reordering fields from the row detail page
better hover effects in the property list
adjustment to overall layout and padding

(Sorry for the potato recording quality, my little toaster can't handle 1080p 60fps)

https://github.com/AppFlowy-IO/AppFlowy/assets/71320345/d815577a-3a75-4748-8e6d-86d5e4d79925

PR Checklist

My code adheres to the AppFlowy Style Guide
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/3328 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 9/4/2023 **Status:** ✅ Merged **Merged:** 9/13/2023 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `row-detail-page-revamp` --- ### 📝 Commits (10+) - [`5719e40`](https://github.com/AppFlowy-IO/AppFlowy/commit/5719e4090430c42b97fb70c209e9611b674df359) feat: revamp row detail page UI - [`2487044`](https://github.com/AppFlowy-IO/AppFlowy/commit/24870440c256abeb470d227a081345d46796e04a) chore: some minor details and fix tests - [`ecb1eda`](https://github.com/AppFlowy-IO/AppFlowy/commit/ecb1eda376e2ade912507a5dd4c4fc6f008f7652) fix: fix tests - [`f9057e5`](https://github.com/AppFlowy-IO/AppFlowy/commit/f9057e514d34b570e57144f8d74e426efa010752) chore: remove unused field - [`c980c28`](https://github.com/AppFlowy-IO/AppFlowy/commit/c980c2834f79d7870228fd6466d89bb3ab46aa52) chore: code cleanup - [`b666a82`](https://github.com/AppFlowy-IO/AppFlowy/commit/b666a8214a3074f919a5d2d4f446e4208000b98d) test: add reordering fields in row page tests - [`b90d68c`](https://github.com/AppFlowy-IO/AppFlowy/commit/b90d68c964b81bff08c88e9ea4755c3f67eff14f) chore: remove duplicate and delete row events - [`2ac985b`](https://github.com/AppFlowy-IO/AppFlowy/commit/2ac985bfb38471f3a533c30a91145ec052b835b8) chore: merge branch 'main' into row-detail-page-revamp - [`2672400`](https://github.com/AppFlowy-IO/AppFlowy/commit/26724006f92467c97e455a35aebf992e6d659e42) chore: timestamp cell ui adjustment - [`0ccfdeb`](https://github.com/AppFlowy-IO/AppFlowy/commit/0ccfdebc7cfeb89fc053267ce885f53ce1b5cd53) chore: remove unused code ### 📊 Changes **23 files changed** (+648 additions, -487 deletions) <details> <summary>View changed files</summary> 📝 `frontend/appflowy_flutter/integration_test/database_calendar_test.dart` (+4 -1) 📝 `frontend/appflowy_flutter/integration_test/database_row_page_test.dart` (+37 -2) 📝 `frontend/appflowy_flutter/integration_test/util/database_test_op.dart` (+43 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/database_controller.dart` (+1 -7) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/row/row_banner_bloc.dart` (+1 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/application/row/row_detail_bloc.dart` (+31 -19) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_cell.dart` (+6 -2) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/row/action.dart` (+2 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/accessory/cell_accessory.dart` (+22 -57) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cell_builder.dart` (+2 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/checkbox_cell/checkbox_cell.dart` (+20 -2) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/date_cell/date_cell.dart` (+28 -11) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/number_cell/number_cell.dart` (+24 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/select_option_cell/extension.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/select_option_cell/select_option_cell.dart` (+10 -8) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/text_cell/text_cell.dart` (+7 -4) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/cells/timestamp_cell/timestamp_cell.dart` (+26 -33) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_action.dart` (+30 -113) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_banner.dart` (+113 -62) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_detail.dart` (+20 -114) _...and 3 more files_ </details> ### 📄 Description Revamp the row detail page. Highlights include: - move duplicate and delete row actions to a sub-menu at the page's top-right corner - allow reordering fields from the row detail page - better hover effects in the property list - adjustment to overall layout and padding (Sorry for the potato recording quality, my little toaster can't handle 1080p 60fps) https://github.com/AppFlowy-IO/AppFlowy/assets/71320345/d815577a-3a75-4748-8e6d-86d5e4d79925 #### PR Checklist - [x] My code adheres to the [AppFlowy Style Guide](https://appflowy.gitbook.io/docs/essential-documentation/contribute-to-appflowy/software-contributions/submitting-code/style-guides) - [ ] 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>