feat(editor): allow auto deciding single side length

This commit added permissive image size auto-deciding process.
Allow single size to be `0` and auto decided when rendering.

Author: realth000

lib/features/editor/widgets/image_dialog.dart

@@ -14,6 +14,7 @@ import 'package:tsdm_client/shared/providers/image_cache_provider/image_cache_pr
 import 'package:tsdm_client/shared/providers/image_cache_provider/models/models.dart';
 import 'package:tsdm_client/utils/logger.dart';
 import 'package:tsdm_client/widgets/section_switch_list_tile.dart';
+import 'package:tsdm_client/widgets/tips.dart';
 
 /// Show a picture dialog to add picture into editor.
 Future<BBCodeImageInfo?> showImagePicker(BuildContext context, {String? url, int? width, int? height}) async =>
@@ -72,6 +73,20 @@ class _ImageDialogState extends State<_ImageDialog> with LoggerMixin, SingleTick
   /// So fill with original image size is fine.
   bool autoFillSize = true;
 
+  /// Automatically determine the width.
+  ///
+  /// Fill a zero value in bbcode.
+  ///
+  /// This can not be true when [autoScaleHeight] is true.
+  bool autoScaleWidth = false;
+
+  /// Automatically determine the height.
+  ///
+  /// Fill a zero value in bbcode.
+  ///
+  /// This can not be true when [autoScaleWidth] is true.
+  bool autoScaleHeight = false;
+
   /// Flag indicating in auto-fill-size progress.
   bool fillingSize = false;
 
@@ -284,41 +299,94 @@ class _ImageDialogState extends State<_ImageDialog> with LoggerMixin, SingleTick
                 // _buildSmmsField(context),
               ],
             ),
-            TextFormField(
-              controller: widthController,
-              keyboardType: TextInputType.number,
-              inputFormatters: [FilteringTextInputFormatter.allow(RegExp('[0-9]+'))],
-              decoration: InputDecoration(
-                prefixIcon: const Icon(Icons.horizontal_distribute_outlined),
-                labelText: tr.width,
-              ),
-              validator: (v) {
-                if (v == null || v.trim().isEmpty) {
-                  return tr.errorEmpty;
-                }
-                final vv = double.tryParse(v);
-                if (vv == null || vv <= 0) {
-                  return tr.errorInvalidNumber;
-                }
-                return null;
-              },
+            Row(
+              children: [
+                Expanded(
+                  child: TextFormField(
+                    controller: widthController,
+                    enabled: !autoScaleWidth,
+                    keyboardType: TextInputType.number,
+                    inputFormatters: [FilteringTextInputFormatter.allow(RegExp('[0-9]+'))],
+                    decoration: InputDecoration(
+                      prefixIcon: const Icon(Icons.horizontal_distribute_outlined),
+                      labelText: tr.width,
+                    ),
+                    validator: (v) {
+                      if (autoScaleWidth) {
+                        return null;
+                      }
+                      if (v == null || v.trim().isEmpty) {
+                        return tr.errorEmpty;
+                      }
+                      final vv = double.tryParse(v);
+                      if (vv == null || vv <= 0) {
+                        return tr.errorInvalidNumber;
+                      }
+                      return null;
+                    },
+                  ),
+                ),
+                Checkbox(
+                  value: autoScaleWidth,
+                  onChanged: (v) {
+                    if (v == null) {
+                      return;
+                    }
+
+                    setState(() {
+                      if (autoScaleHeight) {
+                        autoScaleHeight = false;
+                      }
+                      autoScaleWidth = v;
+                    });
+                  },
+                ),
+                Text(tr.auto),
+              ],
             ),
-            TextFormField(
-              controller: heightController,
-              keyboardType: TextInputType.number,
-              inputFormatters: [FilteringTextInputFormatter.allow(RegExp('[0-9]+'))],
-              decoration: InputDecoration(prefixIcon: const Icon(Icons.add), labelText: tr.height),
-              validator: (v) {
-                if (v == null || v.trim().isEmpty) {
-                  return tr.errorEmpty;
-                }
-                final vv = double.tryParse(v);
-                if (vv == null || vv <= 0) {
-                  return tr.errorInvalidNumber;
-                }
-                return null;
-              },
+            Row(
+              children: [
+                Expanded(
+                  child: TextFormField(
+                    controller: heightController,
+                    enabled: !autoScaleHeight,
+                    keyboardType: TextInputType.number,
+                    inputFormatters: [FilteringTextInputFormatter.allow(RegExp('[0-9]+'))],
+                    decoration: InputDecoration(prefixIcon: const Icon(Icons.add), labelText: tr.height),
+                    validator: (v) {
+                      if (autoScaleHeight) {
+                        return null;
+                      }
+                      if (v == null || v.trim().isEmpty) {
+                        return tr.errorEmpty;
+                      }
+                      final vv = double.tryParse(v);
+                      if (vv == null || vv <= 0) {
+                        return tr.errorInvalidNumber;
+                      }
+                      return null;
+                    },
+                  ),
+                ),
+                Checkbox(
+                  value: autoScaleHeight,
+                  onChanged: (v) {
+                    if (v == null) {
+                      return;
+                    }
+
+                    setState(() {
+                      if (autoScaleWidth) {
+                        autoScaleWidth = false;
+                      }
+                      autoScaleHeight = v;
+                    });
+                  },
+                ),
+                Text(tr.auto),
+              ],
             ),
+            Tips(tr.autoSingleDirectionSize),
             SectionSwitchListTile(
               title: Text(tr.autoFillSize),
               subtitle: Text(tr.autoFillSizeDetail),
@@ -342,12 +410,16 @@ class _ImageDialogState extends State<_ImageDialog> with LoggerMixin, SingleTick
               return;
             }
 
-            final width = int.parse(widthController.text);
-            final height = int.parse(heightController.text);
-            assert(width != 0, 'image width should >= zero');
-            assert(height != 0, 'image height should >= zero');
+            final width = int.tryParse(widthController.text);
+            final height = int.tryParse(heightController.text);
 
-            context.pop(BBCodeImageInfo(urlController.text, width: width, height: height));
+            context.pop(
+              BBCodeImageInfo(
+                urlController.text,
+                width: autoScaleWidth ? 0 : width,
+                height: autoScaleHeight ? 0 : height,
+              ),
+            );
           },
         ),
       ],

lib/features/editor/widgets/rich_editor.dart

@@ -60,16 +60,35 @@ class RichEditor extends StatelessWidget {
       autoFocus: autoFocus,
       scrollController: scrollController,
       imageProvider: (context, url, width, height) {
+        // Requirements:
+        //
+        // 1. If width is not larger than max width, keep the original width and height.
+        // 2. If width is larger than max width, set width to max width and scale height down to the same ratio.
+        // 3. Width and height can not be 0 at the same time.
+
         final w = width?.toDouble();
         final h = height?.toDouble();
         double? maxHeight;
-        if (w != null && h != null && w > _imageMaxWidth) {
-          maxHeight = h * (_imageMaxWidth / w);
+        if (w != null && h != null) {
+          if (w == 0) {
+            // Auto width, do not limit max height.
+            maxHeight = h;
+          } else if (w > _imageMaxWidth && h != 0) {
+            // Width too large, it will be set to max allowed width, scale down the height.
+            maxHeight = h * (_imageMaxWidth / w);
+          } else if (h == 0) {
+            // Auto height.
+            maxHeight = double.infinity;
+          } else {
+            // Normal height.
+            maxHeight = h;
+          }
         }
+
         return CachedImage(
           url,
-          width: w,
-          height: maxHeight == null ? h : null,
+          width: (w == null || w <= 0) ? null : w,
+          height: maxHeight == null ? maxHeight : null,
           maxWidth: _imageMaxWidth,
           maxHeight: maxHeight,
         );

lib/features/notification/view/notification_page.dart

@@ -3,13 +3,11 @@ import 'dart:math' as math;
 import 'package:easy_refresh/easy_refresh.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
-import 'package:go_router/go_router.dart';
 import 'package:tsdm_client/constants/layout.dart';
 import 'package:tsdm_client/features/notification/bloc/auto_notification_cubit.dart';
 import 'package:tsdm_client/features/notification/bloc/notification_bloc.dart';
 import 'package:tsdm_client/features/notification/bloc/notification_state_cubit.dart';
 import 'package:tsdm_client/i18n/strings.g.dart';
-import 'package:tsdm_client/routes/screen_paths.dart';
 import 'package:tsdm_client/shared/models/notification_type.dart';
 import 'package:tsdm_client/utils/logger.dart';
 import 'package:tsdm_client/utils/retry_button.dart';

lib/i18n/en.i18n.json

@@ -704,7 +704,9 @@
       "height": "Height",
       "errorInvalidNumber": "Invalid number",
       "autoFillSize": "Auto fill size",
-      "autoFillSizeDetail": "Fill size with image real size"
+      "autoFillSizeDetail": "Fill size with image real size",
+      "auto": "Auto",
+      "autoSingleDirectionSize": "If would like a auto decided width or height according to image size, check boxes right to width or height above."
     },
     "userMention": {
       "title": "Mention Someone",

lib/i18n/zh-CN.i18n.json

@@ -704,7 +704,9 @@
       "height": "高度",
       "errorInvalidNumber": "无效的数字",
       "autoFillSize": "自动填写大小",
-      "autoFillSizeDetail": "填写图片的实际大小"
+      "autoFillSizeDetail": "填写图片的实际大小",
+      "auto": "自动",
+      "autoSingleDirectionSize": "如果想根据图片大小比例自动设置图片的长或宽，勾选长或宽右边的方框"
     },
     "userMention": {
       "title": "提醒用户（@）",

lib/i18n/zh-TW.i18n.json

@@ -704,7 +704,9 @@
       "height": "高度",
       "errorInvalidNumber": "無效的數字",
       "autoFillSize": "自動填入大小",
-      "autoFillSizeDetail": "填寫圖片的實際大小"
+      "autoFillSizeDetail": "填寫圖片的實際大小",
+      "auto": "自動",
+      "autoSingleDirectionSize": "如果想根據圖片大小比例自動設定圖片的長或寬，勾選長或寬右邊的方框"
     },
     "userMention": {
       "title": "提醒使用者（@）",