feat: sync the icon & title (AppFlowy-IO#4150)

* feat: sync the icon & title * feat: diff the view data when refreshing * fix: unable to update folder * test: refactor bloc tests
AnsahMohammad · Dec 20, 2023 · 9a1ea13 · 9a1ea13
1 parent b466b42
commit 9a1ea13
Show file tree

Hide file tree

Showing 24 changed files with 374 additions and 785 deletions.
diff --git a/frontend/appflowy_flutter/integration_test/util/expectation.dart b/frontend/appflowy_flutter/integration_test/util/expectation.dart
@@ -202,7 +202,7 @@ extension Expectation on WidgetTester {
     return find.descendant(
       of: find.byWidgetPredicate(
         (widget) =>
-            widget is ViewItem &&
+            widget is InnerViewItem &&
             widget.view.name == parentName &&
             widget.view.layout == parentLayout,
         skipOffstage: false,

diff --git a/...owy_flutter/lib/mobile/presentation/home/personal_folder/mobile_home_personal_folder.dart b/...owy_flutter/lib/mobile/presentation/home/personal_folder/mobile_home_personal_folder.dart
@@ -45,7 +45,7 @@ class MobilePersonalFolder extends StatelessWidget {
                 ...views.map(
                   (view) => MobileViewItem(
                     key: ValueKey(
-                      '${FolderCategoryType.personal.name} ${view.hashCode} ',
+                      '${FolderCategoryType.personal.name} ${view.id}',
                     ),
                     isDraggable: true,
                     categoryType: FolderCategoryType.personal,

diff --git a/frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart b/frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart
@@ -76,14 +76,10 @@ class MobileViewItem extends StatelessWidget {
             p.lastCreatedView?.id != c.lastCreatedView!.id,
         listener: (context, state) => context.pushView(state.lastCreatedView!),
         builder: (context, state) {
-          // don't remove this code. it's related to the backend service.
-          view.childViews
-            ..clear()
-            ..addAll(state.childViews);
           return InnerMobileViewItem(
             view: state.view,
             parentView: parentView,
-            childViews: state.childViews,
+            childViews: state.view.childViews,
             categoryType: categoryType,
             level: level,
             leftPadding: leftPadding,
@@ -163,7 +159,7 @@ class InnerMobileViewItem extends StatelessWidget {
       if (childViews.isNotEmpty) {
         final children = childViews.map((childView) {
           return MobileViewItem(
-            key: ValueKey('${categoryType.name} ${childView.hashCode}'),
+            key: ValueKey('${categoryType.name} ${childView.id}'),
             parentView: view,
             categoryType: categoryType,
             isFirstChild: childView.id == childViews.first.id,

diff --git a/frontend/appflowy_flutter/lib/workspace/application/app/app_bloc.dart b/frontend/appflowy_flutter/lib/workspace/application/app/app_bloc.dart
diff --git a/frontend/appflowy_flutter/lib/workspace/application/app/prelude.dart b/frontend/appflowy_flutter/lib/workspace/application/app/prelude.dart