♿ Optimize and fixes

lib/router.dart

@@ -104,7 +104,6 @@ abstract class AppRouter {
               reply: arguments?.reply,
               repost: arguments?.repost,
               realm: arguments?.realm,
-              postListController: arguments?.postListController,
               mode: int.tryParse(state.uri.queryParameters['mode'] ?? '0') ?? 0,
             ),
             transitionsBuilder:

lib/screens/account/profile_page.dart

@@ -300,6 +300,7 @@ class _AccountProfilePageState extends State<AccountProfilePage> {
                     PostWarpedListWidget(
                       isPinned: false,
                       controller: _postController.pagingController,
+                      onUpdate: () => _postController.reloadAllOver(),
                     ),
                 ]),
               ),

lib/screens/feed/draft_box.dart

@@ -76,6 +76,8 @@ class _DraftBoxScreenState extends State<DraftBoxScreen> {
               itemBuilder: (context, item, index) {
                 return PostOwnedListEntry(
                   item: item,
+                  backgroundColor:
+                      Theme.of(context).colorScheme.surfaceContainerLow,
                   onTap: () async {
                     showModalBottomSheet(
                       useRootNavigator: true,
@@ -85,7 +87,13 @@ class _DraftBoxScreenState extends State<DraftBoxScreen> {
                         noReact: true,
                       ),
                     ).then((value) {
-                      if (value != null) _pagingController.refresh();
+                      if (value is Future) {
+                        value.then((_) {
+                          _pagingController.refresh();
+                        });
+                      } else if (value != null) {
+                        _pagingController.refresh();
+                      }
                     });
                   },
                 ).paddingOnly(left: 12, right: 12, bottom: 4);

lib/screens/feed/search.dart

@@ -77,7 +77,10 @@ class _FeedSearchScreenState extends State<FeedSearchScreen> {
                 onRefresh: () => Future.sync(() => _pagingController.refresh()),
                 child: CustomScrollView(
                   slivers: [
-                    PostWarpedListWidget(controller: _pagingController),
+                    PostWarpedListWidget(
+                      controller: _pagingController,
+                      onUpdate: () => _pagingController.refresh(),
+                    ),
                   ],
                 ),
               ),

lib/screens/home.dart

@@ -4,7 +4,6 @@ import 'package:solian/controllers/post_list_controller.dart';
 import 'package:solian/providers/auth.dart';
 import 'package:solian/router.dart';
 import 'package:solian/screens/account/notification.dart';
-import 'package:solian/screens/posts/post_editor.dart';
 import 'package:solian/theme.dart';
 import 'package:solian/widgets/app_bar_title.dart';
 import 'package:solian/widgets/current_state_action.dart';
@@ -47,15 +46,20 @@ class _HomeScreenState extends State<HomeScreen>
       child: Scaffold(
         floatingActionButton: FloatingActionButton(
           child: const Icon(Icons.add),
-          onPressed: () {
+          onPressed: () async {
-            showModalBottomSheet(
+            final value = await showModalBottomSheet(
               useRootNavigator: true,
               isScrollControlled: true,
               context: context,
-              builder: (context) => PostCreatePopup(
+              builder: (context) => const PostCreatePopup(),
-                controller: _postController,
-              ),
             );
+            if (value is Future) {
+              value.then((_) {
+                _postController.reloadAllOver();
+              });
+            } else if (value != null) {
+              _postController.reloadAllOver();
+            }
           },
         ),
         body: NestedScrollView(
@@ -100,6 +104,7 @@ class _HomeScreenState extends State<HomeScreen>
                   child: CustomScrollView(slivers: [
                     PostWarpedListWidget(
                       controller: _postController.pagingController,
+                      onUpdate: () => _postController.reloadAllOver(),
                     ),
                   ]),
                 ),
@@ -121,12 +126,10 @@ class _HomeScreenState extends State<HomeScreen>
 
 class PostCreatePopup extends StatelessWidget {
   final bool hideDraftBox;
-  final PostListController controller;
 
   const PostCreatePopup({
     super.key,
     this.hideDraftBox = false,
-    required this.controller,
   });
 
   @override
@@ -142,13 +145,14 @@ class PostCreatePopup extends StatelessWidget {
         icon: const Icon(Icons.post_add),
         label: 'postEditorModeStory'.tr,
         onTap: () {
-          Navigator.pop(context);
+          Navigator.pop(
-          AppRouter.instance.pushNamed(
+            context,
-            'postEditor',
+            AppRouter.instance.pushNamed(
-            extra: PostPublishArguments(postListController: controller),
+              'postEditor',
-            queryParameters: {
+              queryParameters: {
-              'mode': 0.toString(),
+                'mode': 0.toString(),
-            },
+              },
+            ),
           );
         },
       ),
@@ -156,13 +160,14 @@ class PostCreatePopup extends StatelessWidget {
         icon: const Icon(Icons.description),
         label: 'postEditorModeArticle'.tr,
         onTap: () {
-          Navigator.pop(context);
+          Navigator.pop(
-          AppRouter.instance.pushNamed(
+            context,
-            'postEditor',
+            AppRouter.instance.pushNamed(
-            extra: PostPublishArguments(postListController: controller),
+              'postEditor',
-            queryParameters: {
+              queryParameters: {
-              'mode': 1.toString(),
+                'mode': 1.toString(),
-            },
+              },
+            ),
           );
         },
       ),
@@ -170,8 +175,10 @@ class PostCreatePopup extends StatelessWidget {
         icon: const Icon(Icons.drafts),
         label: 'draftBoxOpen'.tr,
         onTap: () {
-          Navigator.pop(context);
+          Navigator.pop(
-          AppRouter.instance.pushNamed('draftBox');
+            context,
+            AppRouter.instance.pushNamed('draftBox'),
+          );
         },
       ),
     ];

lib/screens/posts/post_detail.dart

@@ -61,6 +61,7 @@ class _PostDetailScreenState extends State<PostDetailScreen> {
                   item: item!,
                   isClickable: true,
                   isFullDate: true,
+                  isFullContent: true,
                   isShowReply: false,
                   isContentSelectable: true,
                 ),

lib/screens/posts/post_editor.dart

@@ -5,7 +5,6 @@ import 'package:get/get.dart';
 import 'package:intl/intl.dart';
 import 'package:markdown_toolbar/markdown_toolbar.dart';
 import 'package:solian/controllers/post_editor_controller.dart';
-import 'package:solian/controllers/post_list_controller.dart';
 import 'package:solian/exts.dart';
 import 'package:solian/models/post.dart';
 import 'package:solian/models/realm.dart';
@@ -23,14 +22,12 @@ class PostPublishArguments {
   final Post? reply;
   final Post? repost;
   final Realm? realm;
-  final PostListController? postListController;
 
   PostPublishArguments({
     this.edit,
     this.reply,
     this.repost,
     this.realm,
-    this.postListController,
   });
 }
 
@@ -39,7 +36,6 @@ class PostPublishScreen extends StatefulWidget {
   final Post? reply;
   final Post? repost;
   final Realm? realm;
-  final PostListController? postListController;
   final int mode;
 
   const PostPublishScreen({
@@ -48,7 +44,6 @@ class PostPublishScreen extends StatefulWidget {
     this.reply,
     this.repost,
     this.realm,
-    this.postListController,
     required this.mode,
   });
 
@@ -95,9 +90,6 @@ class _PostPublishScreenState extends State<PostPublishScreen> {
       context.showErrorDialog(resp.bodyString);
     } else {
       _editorController.localClear();
-      if (widget.postListController != null) {
-        widget.postListController!.reloadAllOver();
-      }
       AppRouter.instance.pop(resp.body);
     }
 

lib/translations/en_us.dart

@@ -361,4 +361,5 @@ const i18nEnglish = {
   'stickerUploaderName': 'Name',
   'stickerUploaderNameHint':
       'A human-friendly name given to the user in the sticker selection interface.',
+  'readMore': 'Read more',
 };

lib/translations/zh_cn.dart

@@ -329,4 +329,5 @@ const i18nSimplifiedChinese = {
   'stickerUploaderAliasHint': '将会在输入时使用和贴图包前缀组成占位符。',
   'stickerUploaderName': '贴图名称',
   'stickerUploaderNameHint': '在贴图选择界面提供给用户的人类友好名称。',
+  'readMore': '阅读更多',
 };

lib/widgets/account/account_profile_popup.dart

@@ -1,6 +1,5 @@
 import 'package:flutter/material.dart';
 import 'package:get/get.dart';
-import 'package:solian/exts.dart';
 import 'package:solian/models/account.dart';
 import 'package:solian/providers/account_status.dart';
 import 'package:solian/router.dart';
@@ -18,23 +17,38 @@ class AccountProfilePopup extends StatefulWidget {
 
 class _AccountProfilePopupState extends State<AccountProfilePopup> {
   bool _isBusy = true;
+  dynamic _hasError;
 
   Account? _userinfo;
 
   void _getUserinfo() async {
     setState(() => _isBusy = true);
 
-    final client = ServiceFinder.configureClient('auth');
+    try {
-    final resp = await client.get('/users/${widget.name}');
+      final client = ServiceFinder.configureClient('auth');
-    if (resp.statusCode == 200) {
+      final resp = await client.get('/users/${widget.name}');
-      _userinfo = Account.fromJson(resp.body);
+      if (resp.statusCode == 200) {
-      setState(() => _isBusy = false);
+        setState(() {
-    } else {
+          _userinfo = Account.fromJson(resp.body);
-      context.showErrorDialog(resp.bodyString);
+          _isBusy = false;
-      Navigator.pop(context);
+        });
+      } else {
+        setState(() {
+          _hasError = resp.bodyString;
+          _isBusy = false;
+        });
+      }
+    } catch (e) {
+      setState(() {
+        _hasError = e;
+        _isBusy = false;
+      });
     }
   }
 
+  Color get _unFocusColor =>
+      Theme.of(context).colorScheme.onSurface.withOpacity(0.75);
+
   @override
   void initState() {
     super.initState();
@@ -43,13 +57,35 @@ class _AccountProfilePopupState extends State<AccountProfilePopup> {
 
   @override
   Widget build(BuildContext context) {
-    if (_isBusy || _userinfo == null) {
+    if (_isBusy) {
       return SizedBox(
         height: MediaQuery.of(context).size.height * 0.75,
         child: const Center(child: CircularProgressIndicator()),
       );
     }
 
+    if (_hasError != null) {
+      return SizedBox(
+        height: MediaQuery.of(context).size.height * 0.75,
+        width: double.infinity,
+        child: Column(
+          mainAxisAlignment: MainAxisAlignment.center,
+          children: [
+            const Icon(Icons.cancel, size: 24),
+            const SizedBox(height: 12),
+            Text(
+              _hasError.toString(),
+              textAlign: TextAlign.center,
+              style: TextStyle(
+                fontSize: 13,
+                color: _unFocusColor,
+              ),
+            ),
+          ],
+        ),
+      );
+    }
+
     return SizedBox(
       height: MediaQuery.of(context).size.height * 0.75,
       child: Column(

lib/widgets/posts/post_action.dart

@@ -97,13 +97,13 @@ class _PostActionState extends State<PostAction> {
                     leading: const FaIcon(FontAwesomeIcons.reply, size: 20),
                     title: Text('reply'.tr),
                     onTap: () async {
-                      final value = await AppRouter.instance.pushNamed(
+                      Navigator.pop(
-                        'postEditor',
+                        context,
-                        extra: PostPublishArguments(reply: widget.item),
+                        AppRouter.instance.pushNamed(
+                          'postEditor',
+                          extra: PostPublishArguments(reply: widget.item),
+                        ),
                       );
-                      if (value != null) {
-                        Navigator.pop(context, true);
-                      }
                     },
                   ),
                 if (!widget.noReact)
@@ -112,13 +112,13 @@ class _PostActionState extends State<PostAction> {
                     leading: const FaIcon(FontAwesomeIcons.retweet, size: 20),
                     title: Text('repost'.tr),
                     onTap: () async {
-                      final value = await AppRouter.instance.pushNamed(
+                      Navigator.pop(
-                        'postEditor',
+                        context,
-                        extra: PostPublishArguments(repost: widget.item),
+                        AppRouter.instance.pushNamed(
+                          'postEditor',
+                          extra: PostPublishArguments(repost: widget.item),
+                        ),
                       );
-                      if (value != null) {
-                        Navigator.pop(context, true);
-                      }
                     },
                   ),
                 if (_canModifyContent && !widget.noReact)
@@ -146,13 +146,13 @@ class _PostActionState extends State<PostAction> {
                     leading: const Icon(Icons.edit),
                     title: Text('edit'.tr),
                     onTap: () async {
-                      final value = await AppRouter.instance.pushNamed(
+                      Navigator.pop(
-                        'postEditor',
+                        context,
-                        extra: PostPublishArguments(edit: widget.item),
+                        AppRouter.instance.pushNamed(
+                          'postEditor',
+                          extra: PostPublishArguments(edit: widget.item),
+                        ),
                       );
-                      if (value != null) {
-                        Navigator.pop(context, true);
-                      }
                     },
                   ),
                 if (_canModifyContent)

lib/widgets/posts/post_item.dart

@@ -1,5 +1,6 @@
 import 'package:animations/animations.dart';
 import 'package:flutter/material.dart';
+import 'package:flutter/rendering.dart';
 import 'package:font_awesome_flutter/font_awesome_flutter.dart';
 import 'package:get/get_utils/get_utils.dart';
 import 'package:intl/intl.dart';
@@ -23,6 +24,7 @@ class PostItem extends StatefulWidget {
   final bool isShowReply;
   final bool isShowEmbed;
   final bool isFullDate;
+  final bool isFullContent;
   final bool isContentSelectable;
   final String? attachmentParent;
   final Color? backgroundColor;
@@ -36,6 +38,7 @@ class PostItem extends StatefulWidget {
     this.isShowReply = true,
     this.isShowEmbed = true,
     this.isFullDate = false,
+    this.isFullContent = false,
     this.isContentSelectable = false,
     this.attachmentParent,
     this.backgroundColor,
@@ -164,6 +167,7 @@ class _PostItemState extends State<PostItem> {
 
   Widget _buildReply(BuildContext context) {
     return OpenContainer(
+      tappable: widget.isClickable,
       closedBuilder: (_, openContainer) => Column(
         children: [
           Row(
@@ -202,13 +206,15 @@ class _PostItemState extends State<PostItem> {
       ),
       closedElevation: 0,
       openElevation: 0,
-      closedColor: widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
+      closedColor:
+          widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
       openColor: Theme.of(context).colorScheme.surface,
     );
   }
 
   Widget _buildRepost(BuildContext context) {
     return OpenContainer(
+      tappable: widget.isClickable,
       closedBuilder: (_, openContainer) => Column(
         children: [
           Row(
@@ -247,11 +253,14 @@ class _PostItemState extends State<PostItem> {
       ),
       closedElevation: 0,
       openElevation: 0,
-      closedColor: widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
+      closedColor:
+          widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
       openColor: Theme.of(context).colorScheme.surface,
     );
   }
 
+  double _contentHeight = 0;
+
   @override
   Widget build(BuildContext context) {
     final List<int> attachments = item.body['attachments'] is List
@@ -295,6 +304,7 @@ class _PostItemState extends State<PostItem> {
     }
 
     return OpenContainer(
+      tappable: widget.isClickable,
       closedBuilder: (_, openContainer) => Column(
         crossAxisAlignment: CrossAxisAlignment.start,
         children: [
@@ -322,11 +332,31 @@ class _PostItemState extends State<PostItem> {
                     _buildHeader(),
                     SizedContainer(
                       maxWidth: 640,
-                      child: MarkdownTextContent(
+                      maxHeight: widget.isFullContent ? double.infinity : 320,
-                        content: item.body['content'],
+                      child: _MeasureSize(
-                        isSelectable: widget.isContentSelectable,
+                        onChange: (size) {
-                      ).paddingOnly(left: 12, right: 8),
+                          _contentHeight = size.height;
+                        },
+                        child: MarkdownTextContent(
+                          content: item.body['content'],
+                          isSelectable: widget.isContentSelectable,
+                        ).paddingOnly(left: 12, right: 8),
+                      ),
                     ),
+                    if (_contentHeight >= 320 &&
+                        widget.isClickable &&
+                        !widget.isFullContent)
+                      InkWell(
+                        child: Text(
+                          'readMore'.tr,
+                          style: TextStyle(
+                            color: Theme.of(context).colorScheme.primary,
+                          ),
+                        ),
+                        onTap: () {
+                          openContainer();
+                        },
+                      ).paddingOnly(left: 12, top: 4),
                     if (widget.item.replyTo != null && widget.isShowEmbed)
                       _buildReply(context).paddingOnly(top: 4),
                     if (widget.item.repostTo != null && widget.isShowEmbed)
@@ -377,8 +407,52 @@ class _PostItemState extends State<PostItem> {
       ),
       closedElevation: 0,
       openElevation: 0,
-      closedColor: widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
+      closedColor:
+          widget.backgroundColor ?? Theme.of(context).colorScheme.surface,
       openColor: Theme.of(context).colorScheme.surface,
     );
   }
 }
+
+typedef _OnWidgetSizeChange = void Function(Size size);
+
+class _MeasureSizeRenderObject extends RenderProxyBox {
+  Size? oldSize;
+  _OnWidgetSizeChange onChange;
+
+  _MeasureSizeRenderObject(this.onChange);
+
+  @override
+  void performLayout() {
+    super.performLayout();
+
+    Size newSize = child!.size;
+    if (oldSize == newSize) return;
+
+    oldSize = newSize;
+    WidgetsBinding.instance.addPostFrameCallback((_) {
+      onChange(newSize);
+    });
+  }
+}
+
+class _MeasureSize extends SingleChildRenderObjectWidget {
+  final _OnWidgetSizeChange onChange;
+
+  const _MeasureSize({
+    super.key,
+    required this.onChange,
+    required Widget super.child,
+  });
+
+  @override
+  RenderObject createRenderObject(BuildContext context) {
+    return _MeasureSizeRenderObject(onChange);
+  }
+
+  @override
+  void updateRenderObject(
+      BuildContext context, covariant _MeasureSizeRenderObject renderObject) {
+    renderObject.onChange = onChange;
+  }
+}

lib/widgets/posts/post_list.dart

@@ -85,7 +85,13 @@ class PostListEntryWidget extends StatelessWidget {
           context: context,
           builder: (context) => PostAction(item: item),
         ).then((value) {
-          if (value != null) onUpdate();
+          if (value is Future) {
+            value.then((_) {
+              onUpdate();
+            });
+          } else if (value != null) {
+            onUpdate();
+          }
         });
       },
     );

lib/widgets/posts/post_owned_list.dart

@@ -6,18 +6,19 @@ import 'package:solian/widgets/posts/post_item.dart';
 class PostOwnedListEntry extends StatelessWidget {
   final Post item;
   final Function onTap;
+  final Color? backgroundColor;
 
   const PostOwnedListEntry({
     super.key,
     required this.item,
     required this.onTap,
+    this.backgroundColor,
   });
 
   @override
   Widget build(BuildContext context) {
     return Card(
-      child: InkWell(
+      child: GestureDetector(
-        borderRadius: const BorderRadius.all(Radius.circular(8)),
         child: Column(
           crossAxisAlignment: CrossAxisAlignment.start,
           children: [
@@ -28,6 +29,7 @@ class PostOwnedListEntry extends StatelessWidget {
               isClickable: false,
               isShowReply: false,
               isReactable: false,
+              backgroundColor: backgroundColor,
             ).paddingSymmetric(vertical: 8),
           ],
         ),

lib/widgets/posts/post_warped_list.dart

@@ -9,6 +9,7 @@ class PostWarpedListWidget extends StatelessWidget {
   final bool isNestedClickable;
   final bool isPinned;
   final PagingController<int, Post> controller;
+  final Function? onUpdate;
 
   const PostWarpedListWidget({
     super.key,
@@ -17,6 +18,7 @@ class PostWarpedListWidget extends StatelessWidget {
     this.isClickable = true,
     this.isNestedClickable = true,
     this.isPinned = true,
+    this.onUpdate,
   });
 
   @override
@@ -35,9 +37,7 @@ class PostWarpedListWidget extends StatelessWidget {
             isNestedClickable: isNestedClickable,
             isClickable: isClickable,
             item: item,
-            onUpdate: () {
+            onUpdate: onUpdate ?? () {},
-              controller.refresh();
-            },
           );
         },
       ),

lib/widgets/sized_container.dart

@@ -3,11 +3,13 @@ import 'package:flutter/material.dart';
 class SizedContainer extends StatelessWidget {
   final Widget child;
   final double maxWidth;
+  final double maxHeight;
 
   const SizedContainer({
     super.key,
     required this.child,
     this.maxWidth = 720,
+    this.maxHeight = double.infinity,
   });
 
   @override
@@ -15,7 +17,7 @@ class SizedContainer extends StatelessWidget {
     return Align(
       alignment: Alignment.centerLeft,
       child: Container(
-        constraints: BoxConstraints(maxWidth: maxWidth),
+        constraints: BoxConstraints(maxWidth: maxWidth, maxHeight: maxHeight),
         child: child,
       ),
     );

pubspec.lock

@@ -385,6 +385,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "2.1.2"
+  field_suggestion:
+    dependency: "direct main"
+    description:
+      name: field_suggestion
+      sha256: "596362ed67a661a18e01e4f51b8b92424f333baf37cebd48cc647a4c82c858d5"
+      url: "https://pub.dev"
+    source: hosted
+    version: "0.2.5"
   file:
     dependency: transitive
     description:
@@ -736,6 +744,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "2.3.2"
+  highlightable:
+    dependency: transitive
+    description:
+      name: highlightable
+      sha256: "526793e148c91977b694d75d99cd34401ea3b65efd223e7b539c76916af86ffd"
+      url: "https://pub.dev"
+    source: hosted
+    version: "1.0.5"
   http:
     dependency: transitive
     description:

pubspec.yaml

@@ -66,6 +66,7 @@ dependencies:
   animations: ^2.0.11
   avatar_stack: ^1.2.0
   async: ^2.11.0
+  field_suggestion: ^0.2.5
 
 dev_dependencies:
   flutter_test: