fix: sidebar issues

frontend/appflowy_flutter/lib/mobile/application/recent/recent_view_bloc.dart

@@ -57,7 +57,19 @@ class RecentViewBloc extends Bloc<RecentViewEvent, RecentViewState> {
                 }
               },
             );
+
+            // only document supports the cover
+            if (view.layout != ViewLayoutPB.Document) {
+              emit(
+                state.copyWith(
+                  name: view.name,
+                  icon: view.icon.value,
+                ),
+              );
+            }
+
             final cover = getCoverV2();
+
             if (cover != null) {
               emit(
                 state.copyWith(

frontend/appflowy_flutter/lib/mobile/presentation/home/favorite_folder/favorite_space.dart

@@ -11,24 +11,36 @@ import 'package:flowy_infra_ui/flowy_infra_ui.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 
-class MobileFavoriteSpace extends StatelessWidget {
+class MobileFavoriteSpace extends StatefulWidget {
   const MobileFavoriteSpace({
     super.key,
     required this.userProfile,
   });
 
   final UserProfilePB userProfile;
 
+  @override
+  State<MobileFavoriteSpace> createState() => _MobileFavoriteSpaceState();
+}
+
+class _MobileFavoriteSpaceState extends State<MobileFavoriteSpace>
+    with AutomaticKeepAliveClientMixin {
+  @override
+  bool get wantKeepAlive => true;
+
   @override
   Widget build(BuildContext context) {
+    super.build(context);
     final workspaceId =
         context.read<UserWorkspaceBloc>().state.currentWorkspace?.workspaceId ??
             '';
     return MultiBlocProvider(
       providers: [
         BlocProvider(
           create: (_) => SidebarSectionsBloc()
-            ..add(SidebarSectionsEvent.initial(userProfile, workspaceId)),
+            ..add(
+              SidebarSectionsEvent.initial(widget.userProfile, workspaceId),
+            ),
         ),
         BlocProvider(
           create: (_) => FavoriteBloc()..add(const FavoriteEvent.initial()),

frontend/appflowy_flutter/lib/mobile/presentation/home/home_space/home_space.dart

@@ -5,13 +5,23 @@ import 'package:appflowy_backend/protobuf/flowy-user/protobuf.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 
-class MobileHomeSpace extends StatelessWidget {
+class MobileHomeSpace extends StatefulWidget {
   const MobileHomeSpace({super.key, required this.userProfile});
 
   final UserProfilePB userProfile;
 
+  @override
+  State<MobileHomeSpace> createState() => _MobileHomeSpaceState();
+}
+
+class _MobileHomeSpaceState extends State<MobileHomeSpace>
+    with AutomaticKeepAliveClientMixin {
+  @override
+  bool get wantKeepAlive => true;
+
   @override
   Widget build(BuildContext context) {
+    super.build(context);
     final workspaceId =
         context.read<UserWorkspaceBloc>().state.currentWorkspace?.workspaceId ??
             '';
@@ -23,7 +33,7 @@ class MobileHomeSpace extends StatelessWidget {
             vertical: HomeSpaceViewSizes.mVerticalPadding,
           ),
           child: MobileFolders(
-            user: userProfile,
+            user: widget.userProfile,
             workspaceId: workspaceId,
             showFavorite: false,
           ),

frontend/appflowy_flutter/lib/mobile/presentation/home/recent_folder/recent_space.dart

@@ -7,11 +7,21 @@ import 'package:flowy_infra_ui/flowy_infra_ui.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 
-class MobileRecentSpace extends StatelessWidget {
+class MobileRecentSpace extends StatefulWidget {
   const MobileRecentSpace({super.key});
 
+  @override
+  State<MobileRecentSpace> createState() => _MobileRecentSpaceState();
+}
+
+class _MobileRecentSpaceState extends State<MobileRecentSpace>
+    with AutomaticKeepAliveClientMixin {
+  @override
+  bool get wantKeepAlive => true;
+
   @override
   Widget build(BuildContext context) {
+    super.build(context);
     return BlocProvider(
       create: (context) =>
           RecentViewsBloc()..add(const RecentViewsEvent.initial()),

frontend/appflowy_flutter/lib/mobile/presentation/home/shared/mobile_view_card.dart

@@ -53,8 +53,8 @@ class MobileViewCard extends StatelessWidget {
     return MultiBlocProvider(
       providers: [
         BlocProvider<ViewBloc>(
-          create: (context) =>
-              ViewBloc(view: view)..add(const ViewEvent.initial()),
+          create: (context) => ViewBloc(view: view, shouldLoadChildViews: false)
+            ..add(const ViewEvent.initial()),
         ),
         BlocProvider(
           create: (context) =>

frontend/appflowy_flutter/lib/mobile/presentation/home/tab/_tab_bar.dart

@@ -39,6 +39,7 @@ class MobileSpaceTabBar extends StatelessWidget {
         indicatorColor: Theme.of(context).primaryColor,
         isScrollable: true,
         labelStyle: labelStyle,
+        labelColor: baseStyle?.color,
         labelPadding: const EdgeInsets.symmetric(horizontal: 12.0),
         unselectedLabelStyle: unselectedLabelStyle,
         overlayColor: WidgetStateProperty.all(Colors.transparent),

frontend/appflowy_flutter/lib/mobile/presentation/mobile_bottom_navigation_bar.dart

@@ -21,7 +21,7 @@ class MobileBottomNavigationBar extends StatelessWidget {
     return Scaffold(
       body: navigationShell,
       bottomNavigationBar: Theme(
-        data: ThemeData(
+        data: Theme.of(context).copyWith(
           splashColor: Colors.transparent,
           highlightColor: Colors.transparent,
         ),

frontend/appflowy_flutter/lib/mobile/presentation/page_item/mobile_view_item.dart

@@ -257,7 +257,6 @@ class _SingleMobileInnerViewItemState extends State<SingleMobileInnerViewItem> {
     final children = [
       // expand icon
       _buildLeftIcon(),
-      const HSpace(6),
       // icon
       _buildViewIcon(),
       const HSpace(8),
@@ -274,11 +273,11 @@ class _SingleMobileInnerViewItemState extends State<SingleMobileInnerViewItem> {
     // hover action
 
     // ··· more action button
-    // children.add(_buildViewMoreActionButton(context));
+    children.add(_buildViewMoreButton(context));
     // only support add button for document layout
     if (!widget.isFeedback && widget.view.layout == ViewLayoutPB.Document) {
       // + button
-      children.add(_buildViewMoreButton(context));
+
       children.add(_buildViewAddButton(context));
     }
 
@@ -326,22 +325,20 @@ class _SingleMobileInnerViewItemState extends State<SingleMobileInnerViewItem> {
   // show > if the view is expandable.
   // show · if the view can't contain child views.
   Widget _buildLeftIcon() {
-    if (isReferencedDatabaseView(widget.view, widget.parentView)) {
-      return const _DotIconWidget();
-    }
-
     if (context.read<ViewBloc>().state.view.childViews.isEmpty) {
       return HSpace(widget.leftPadding);
     }
 
     return GestureDetector(
-      child: AnimatedRotation(
-        duration: const Duration(milliseconds: 250),
-        turns: widget.isExpanded ? 0 : -0.25,
-        child: const FlowySvg(
-          FlowySvgs.m_expand_s,
-          blendMode: null,
-        ),
+      behavior: HitTestBehavior.opaque,
+      child: Padding(
+        padding: const EdgeInsets.only(right: 6.0, top: 6.0, bottom: 6.0),
+        child: FlowySvg(
+                 widget.isExpanded 
+                   ? FlowySvgs.m_expand_s
+                   : FlowySvgs.m_collapse_s,
+                blendMode: null,
+              ),
       ),
       onTap: () {
         context
@@ -431,25 +428,6 @@ class _SingleMobileInnerViewItemState extends State<SingleMobileInnerViewItem> {
   }
 }
 
-class _DotIconWidget extends StatelessWidget {
-  const _DotIconWidget();
-
-  @override
-  Widget build(BuildContext context) {
-    return Padding(
-      padding: const EdgeInsets.all(6.0),
-      child: Container(
-        width: 4,
-        height: 4,
-        decoration: BoxDecoration(
-          color: Theme.of(context).iconTheme.color,
-          borderRadius: BorderRadius.circular(2),
-        ),
-      ),
-    );
-  }
-}
-
 // workaround: we should use view.isEndPoint or something to check if the view can contain child views. But currently, we don't have that field.
 bool isReferencedDatabaseView(ViewPB view, ViewPB? parentView) {
   if (parentView == null) {

frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/cover/document_immersive_cover.dart

@@ -1,12 +1,14 @@
 import 'dart:io';
 
+import 'package:appflowy/generated/locale_keys.g.dart';
 import 'package:appflowy/mobile/application/page_style/document_page_style_bloc.dart';
-import 'package:appflowy/plugins/base/emoji/emoji_picker_screen.dart';
-import 'package:appflowy/plugins/base/icon/icon_picker.dart';
+import 'package:appflowy/mobile/presentation/bottom_sheet/bottom_sheet.dart';
 import 'package:appflowy/plugins/document/application/prelude.dart';
 import 'package:appflowy/plugins/document/presentation/editor_plugins/base/build_context_extension.dart';
 import 'package:appflowy/plugins/document/presentation/editor_plugins/cover/document_immersive_cover_bloc.dart';
 import 'package:appflowy/plugins/document/presentation/editor_plugins/header/emoji_icon_widget.dart';
+import 'package:appflowy/plugins/document/presentation/editor_plugins/icon/icon_selector.dart';
+import 'package:appflowy/plugins/document/presentation/editor_plugins/page_style/_page_style_icon_bloc.dart';
 import 'package:appflowy/shared/appflowy_network_image.dart';
 import 'package:appflowy/shared/flowy_gradient_colors.dart';
 import 'package:appflowy/shared/google_fonts_extension.dart';
@@ -17,11 +19,12 @@ import 'package:appflowy_backend/protobuf/flowy-folder/view.pb.dart';
 import 'package:appflowy_backend/protobuf/flowy-user/protobuf.dart';
 import 'package:appflowy_editor/appflowy_editor.dart';
 import 'package:auto_size_text_field/auto_size_text_field.dart';
+import 'package:easy_localization/easy_localization.dart';
+import 'package:flowy_infra/theme_extension.dart';
 import 'package:flowy_infra_ui/flowy_infra_ui.dart';
 import 'package:flowy_infra_ui/widget/ignore_parent_gesture.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:go_router/go_router.dart';
 
 double kDocumentCoverHeight = 98.0;
 double kDocumentTitlePadding = 20.0;
@@ -171,12 +174,40 @@ class _DocumentImmersiveCoverState extends State<DocumentImmersiveCover> {
         ),
       ),
       onTap: () async {
-        final result = await context.push<EmojiPickerResult>(
-          MobileEmojiPickerScreen.routeName,
+        final pageStyleIconBloc = PageStyleIconBloc(view: widget.view)
+          ..add(const PageStyleIconEvent.initial());
+        await showMobileBottomSheet(
+          context,
+          showDragHandle: true,
+          showDivider: false,
+          showDoneButton: true,
+          showHeader: true,
+          title: LocaleKeys.titleBar_pageIcon.tr(),
+          backgroundColor: AFThemeExtension.of(context).background,
+          enableDraggableScrollable: true,
+          minChildSize: 0.6,
+          initialChildSize: 0.61,
+          showRemoveButton: true,
+          onRemove: () {
+            pageStyleIconBloc.add(
+              const PageStyleIconEvent.updateIcon('', true),
+            );
+          },
+          scrollableWidgetBuilder: (_, controller) {
+            return BlocProvider.value(
+              value: pageStyleIconBloc,
+              child: Expanded(
+                child: Scrollbar(
+                  controller: controller,
+                  child: IconSelector(
+                    scrollController: controller,
+                  ),
+                ),
+              ),
+            );
+          },
+          builder: (_) => const SizedBox.shrink(),
         );
-        if (result != null && context.mounted) {
-          context.read<ViewBloc>().add(ViewEvent.updateIcon(result.emoji));
-        }
       },
     );
   }