[PR #3296] [MERGED] chore: per-view field visibility UI #5668

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3296
Author: @richardshiue
Created: 8/30/2023
Status: ✅ Merged
Merged: 9/1/2023
Merged by: @appflowy

Base: main ← Head: field-settings-4

📝 Commits (6)

ff66b3f chore: default field settings if not exist
f9a8785 chore: field settings listeners and services
af3ca51 chore: don't need to updateFieldInfos
ba40071 feat: per-view field visibilty UI
a873d7d chore: merge remote-tracking branch 'upstream/main' into field-settings-4
359194d fix: remove unresolved imports

📊 Changes

17 files changed (+241 additions, -110 deletions)

View changed files

➕ frontend/appflowy_flutter/integration_test/database_field_settings_test.dart (+55 -0)
📝 frontend/appflowy_flutter/integration_test/database_field_test.dart (+0 -20)
📝 frontend/appflowy_flutter/integration_test/runner.dart (+2 -0)
📝 frontend/appflowy_flutter/integration_test/util/database_test_op.dart (+52 -0)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_action_sheet_bloc.dart (+16 -3)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_controller.dart (+7 -6)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_service.dart (+0 -5)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/row/row_cache.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/database_view/application/setting/property_bloc.dart (+7 -5)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/application/grid_header_bloc.dart (+11 -7)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/application/row/row_detail_bloc.dart (+27 -9)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_cell.dart (+1 -6)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_editor.dart (+4 -3)
📝 frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/grid_header.dart (+2 -5)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/field/grid_property.dart (+54 -38)
📝 frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_detail.dart (+1 -1)
📝 frontend/rust-lib/flowy-database2/tests/database/mock_data/board_mock_data.rs (+0 -1)

📄 Description

depends on https://github.com/AppFlowy-IO/AppFlowy/pull/3284

Feature Preview

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/3296 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 8/30/2023 **Status:** ✅ Merged **Merged:** 9/1/2023 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `field-settings-4` --- ### 📝 Commits (6) - [`ff66b3f`](https://github.com/AppFlowy-IO/AppFlowy/commit/ff66b3f88edd04518766e9e95c925854637ea40a) chore: default field settings if not exist - [`f9a8785`](https://github.com/AppFlowy-IO/AppFlowy/commit/f9a8785c6c3059cdffd97f8787426c972c13b678) chore: field settings listeners and services - [`af3ca51`](https://github.com/AppFlowy-IO/AppFlowy/commit/af3ca5171e23117a950eb9dae41526eaf1ef9bea) chore: don't need to updateFieldInfos - [`ba40071`](https://github.com/AppFlowy-IO/AppFlowy/commit/ba400716130da4b4b56f422662f2a4f4c17151a6) feat: per-view field visibilty UI - [`a873d7d`](https://github.com/AppFlowy-IO/AppFlowy/commit/a873d7d0668b3e80daa587a0c4276b6915837115) chore: merge remote-tracking branch 'upstream/main' into field-settings-4 - [`359194d`](https://github.com/AppFlowy-IO/AppFlowy/commit/359194d07254a0f096f493624691e5ae6852940a) fix: remove unresolved imports ### 📊 Changes **17 files changed** (+241 additions, -110 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/integration_test/database_field_settings_test.dart` (+55 -0) 📝 `frontend/appflowy_flutter/integration_test/database_field_test.dart` (+0 -20) 📝 `frontend/appflowy_flutter/integration_test/runner.dart` (+2 -0) 📝 `frontend/appflowy_flutter/integration_test/util/database_test_op.dart` (+52 -0) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_action_sheet_bloc.dart` (+16 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_controller.dart` (+7 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/field/field_service.dart` (+0 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/row/row_cache.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/application/setting/property_bloc.dart` (+7 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/application/grid_header_bloc.dart` (+11 -7) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/application/row/row_detail_bloc.dart` (+27 -9) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_cell.dart` (+1 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/field_editor.dart` (+4 -3) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/grid/presentation/widgets/header/grid_header.dart` (+2 -5) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/field/grid_property.dart` (+54 -38) 📝 `frontend/appflowy_flutter/lib/plugins/database_view/widgets/row/row_detail.dart` (+1 -1) 📝 `frontend/rust-lib/flowy-database2/tests/database/mock_data/board_mock_data.rs` (+0 -1) </details> ### 📄 Description depends on https://github.com/AppFlowy-IO/AppFlowy/pull/3284 ### Feature Preview  ---  #### PR Checklist - [ ] 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. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>