[PR #3935] [MERGED] feat: mobile card detail screen #5956

Closed
opened 2026-03-23 22:21:04 +00:00 by mirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/3935
Author: @hyj1204
Created: 11/14/2023
Status: Merged
Merged: 11/21/2023
Merged by: @hyj1204

Base: mainHead: feat/mobile_card_detail


📝 Commits (10+)

  • 1db66f6 feat: add CardDetailScreen and CardPropertyEditScreen
  • 9f38113 feat: add MobileNumberCell and MobileTimestampCell
  • a877aba feat: Add MobileDateCell and MobileCheckboxCell
  • c046517 feat: add MobileFieldTypeOptionEditor
  • d74aa13 feat: add property delete feature in CardPropertyEditScreen
  • 736bedb fix: fix VisibilitySwitch didn't update
  • 11942c4 feat: add MobileCreateRowFieldScreen
  • 5d5e83a chore: localization and improve spacing
  • 24e4083 feat: add TimestampTypeOptionMobileWidget
  • 03c14ba feat: add checkbox and DateTypeOptionMobileWidget

📊 Changes

49 files changed (+3213 additions, -66 deletions)

View changed files

frontend/appflowy_flutter/lib/mobile/presentation/database/card/card.dart (+1 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart (+186 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_create_row_field_screen.dart (+59 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_create_row_field_button.dart (+47 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_field_name_text_field.dart (+45 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_row_property_list.dart (+221 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/widgets.dart (+3 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/card_property_edit_screen.dart (+64 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_editor.dart (+130 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_type_list.dart (+82 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_type_option_editor.dart (+191 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/checkbox.dart (+12 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/checklist.dart (+12 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/date.dart (+88 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/multi_select.dart (+12 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/number.dart (+154 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/rich_text.dart (+12 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/single_select.dart (+13 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/timestamp.dart (+98 -0)
frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/type_option_widget_builder.dart (+9 -0)

...and 29 more files

📄 Description

Feature Preview

Follow up #3905

Unsolved Issue:

The card detail page cannot be rendered when switching back to itself through the navigation bar.

https://github.com/AppFlowy-IO/AppFlowy/assets/14248245/bbb74c45-cc1c-4255-af1d-e206c9094bed

The card detail page access point is inside the board_page.dart _openCard method.
Currently, it relies on RowController to render

image

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/3935 **Author:** [@hyj1204](https://github.com/hyj1204) **Created:** 11/14/2023 **Status:** ✅ Merged **Merged:** 11/21/2023 **Merged by:** [@hyj1204](https://github.com/hyj1204) **Base:** `main` ← **Head:** `feat/mobile_card_detail` --- ### 📝 Commits (10+) - [`1db66f6`](https://github.com/AppFlowy-IO/AppFlowy/commit/1db66f62d06d011dab613cfa72b6073952d06651) feat: add CardDetailScreen and CardPropertyEditScreen - [`9f38113`](https://github.com/AppFlowy-IO/AppFlowy/commit/9f3811310791c45445cded2607610bee4b4244e3) feat: add MobileNumberCell and MobileTimestampCell - [`a877aba`](https://github.com/AppFlowy-IO/AppFlowy/commit/a877ababb6a710dba9dcb1ffd21c27465854238d) feat: Add MobileDateCell and MobileCheckboxCell - [`c046517`](https://github.com/AppFlowy-IO/AppFlowy/commit/c0465176b2d8abdca848b1f6d22aa70a9a4b28d5) feat: add MobileFieldTypeOptionEditor - [`d74aa13`](https://github.com/AppFlowy-IO/AppFlowy/commit/d74aa13f1d405d9e2b1ebefba0a24d9963e99376) feat: add property delete feature in CardPropertyEditScreen - [`736bedb`](https://github.com/AppFlowy-IO/AppFlowy/commit/736bedb1e87cbf092118c8087d5c3ef933f4fce9) fix: fix VisibilitySwitch didn't update - [`11942c4`](https://github.com/AppFlowy-IO/AppFlowy/commit/11942c4fd9889d3e5b3521d7e947ac1e7e0ade9e) feat: add MobileCreateRowFieldScreen - [`5d5e83a`](https://github.com/AppFlowy-IO/AppFlowy/commit/5d5e83af45abdcf8d20392a86d7b127b8e15e33a) chore: localization and improve spacing - [`24e4083`](https://github.com/AppFlowy-IO/AppFlowy/commit/24e408356cf37726dd49b9720664dbdbfd84cd30) feat: add TimestampTypeOptionMobileWidget - [`03c14ba`](https://github.com/AppFlowy-IO/AppFlowy/commit/03c14baf1de30fc30383a406da7da71ec58a1591) feat: add checkbox and DateTypeOptionMobileWidget ### 📊 Changes **49 files changed** (+3213 additions, -66 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card.dart` (+1 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_card_detail_screen.dart` (+186 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/mobile_create_row_field_screen.dart` (+59 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_create_row_field_button.dart` (+47 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_field_name_text_field.dart` (+45 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/mobile_row_property_list.dart` (+221 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_detail/widgets/widgets.dart` (+3 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/card_property_edit_screen.dart` (+64 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_editor.dart` (+130 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_type_list.dart` (+82 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/mobile_field_type_option_editor.dart` (+191 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/checkbox.dart` (+12 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/checklist.dart` (+12 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/date.dart` (+88 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/multi_select.dart` (+12 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/number.dart` (+154 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/rich_text.dart` (+12 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/single_select.dart` (+13 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/timestamp.dart` (+98 -0) ➕ `frontend/appflowy_flutter/lib/mobile/presentation/database/card/card_property_edit/type_option_widget_builder/type_option_widget_builder.dart` (+9 -0) _...and 29 more files_ </details> ### 📄 Description <!--- Thank you for submitting a pull request to AppFlowy. The team will dedicate their best efforts to reviewing and approving your pull request. If you have any questions about the project or feedback for us, please join our [Discord](https://discord.gg/wdjWUXXhtw). --> <!--- If your pull request adds a new feature, please drag and drop a video into this section to showcase what you've done! If not, you may delete this section. --> ### Feature Preview Follow up #3905 **Unsolved Issue:** The card detail page cannot be rendered when switching back to itself through the navigation bar. https://github.com/AppFlowy-IO/AppFlowy/assets/14248245/bbb74c45-cc1c-4255-af1d-e206c9094bed The card detail page access point is inside the `board_page.dart` `_openCard` method. Currently, it relies on RowController to render <img width="898" alt="image" src="https://github.com/AppFlowy-IO/AppFlowy/assets/14248245/c0b8bb97-04eb-4d04-9264-7247f39335bb"> <!--- List at least one issue here that this PR addresses. If it fixes the issue, please use the [fixes](https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests) keyword to close the issue. For example: fixes https://github.com/AppFlowy-IO/AppFlowy/pull/2106 --> --- <!--- Before you mark this PR ready for review, run through this checklist! --> #### 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. - [ ] 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>
mirror 2026-03-23 22:21:04 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AppFlowy-IO/AppFlowy#5956
No description provided.