💄 Optimize chat input a step further

lib/screens/chat/room.dart

@@ -541,7 +541,10 @@ class ChatRoomScreen extends HookConsumerWidget {
     Widget chatMessageListWidget(List<LocalChatMessage> messageList) =>
         SuperListView.builder(
           listController: listController,
-          padding: EdgeInsets.symmetric(vertical: 16),
+          padding: EdgeInsets.only(
+            top: 16,
+            bottom: 96 + MediaQuery.of(context).padding.bottom,
+          ),
           controller: scrollController,
           reverse: true, // Show newest messages at the bottom
           itemCount: messageList.length,
@@ -735,17 +738,15 @@ class ChatRoomScreen extends HookConsumerWidget {
       ),
       body: Stack(
         children: [
-          Column(
-            children: [
-              Expanded(
+          // Messages
+          Positioned.fill(
             child: messages.when(
               data:
                   (messageList) =>
                       messageList.isEmpty
                           ? Center(child: Text('No messages yet'.tr()))
                           : chatMessageListWidget(messageList),
-                  loading:
-                      () => const Center(child: CircularProgressIndicator()),
+              loading: () => const Center(child: CircularProgressIndicator()),
               error:
                   (error, _) => ResponseErrorWidget(
                     error: error,
@@ -753,7 +754,12 @@ class ChatRoomScreen extends HookConsumerWidget {
                   ),
             ),
           ),
-              chatRoom.when(
+          // Input
+          Positioned(
+            bottom: 0,
+            left: 0,
+            right: 0,
+            child: chatRoom.when(
               data:
                   (room) => Column(
                     mainAxisSize: MainAxisSize.min,
@@ -880,12 +886,12 @@ class ChatRoomScreen extends HookConsumerWidget {
                         },
                         attachmentProgress: attachmentProgress.value,
                       ),
+                      Gap(MediaQuery.of(context).padding.bottom),
                     ],
                   ),
               error: (_, _) => const SizedBox.shrink(),
               loading: () => const SizedBox.shrink(),
             ),
-            ],
           ),
           Positioned(
             left: 0,

lib/widgets/chat/chat_input.dart

@@ -1,7 +1,5 @@
 import "dart:async";
-import "dart:io";
 import "package:easy_localization/easy_localization.dart";
-import "package:flutter/foundation.dart";
 import "package:flutter/material.dart";
 import "package:flutter/services.dart";
 import "package:flutter_hooks/flutter_hooks.dart";
@@ -56,10 +54,6 @@ class ChatInput extends HookConsumerWidget {
   Widget build(BuildContext context, WidgetRef ref) {
     final inputFocusNode = useFocusNode();
 
-    final enterToSend = ref.watch(appSettingsNotifierProvider).enterToSend;
-
-    final isMobile = !kIsWeb && (Platform.isAndroid || Platform.isIOS);
-
     void send() {
       onSend.call();
       WidgetsBinding.instance.addPostFrameCallback((_) {
@@ -67,6 +61,18 @@ class ChatInput extends HookConsumerWidget {
       });
     }
 
+    void insertNewLine() {
+      final text = messageController.text;
+      final selection = messageController.selection;
+      final start = selection.start >= 0 ? selection.start : text.length;
+      final end = selection.end >= 0 ? selection.end : text.length;
+      final newText = text.replaceRange(start, end, '\n');
+      messageController.value = TextEditingValue(
+        text: newText,
+        selection: TextSelection.collapsed(offset: start + 1),
+      );
+    }
+
     Future<void> handlePaste() async {
       final clipboard = await Pasteboard.image;
       if (clipboard == null) return;
@@ -80,36 +86,43 @@ class ChatInput extends HookConsumerWidget {
       ]);
     }
 
-    void handleKeyPress(
-      BuildContext context,
-      WidgetRef ref,
-      RawKeyEvent event,
-    ) {
-      if (event is! RawKeyDownEvent) return;
+    inputFocusNode.onKeyEvent = (node, event) {
+      if (event is! KeyDownEvent) return KeyEventResult.ignored;
 
       final isPaste = event.logicalKey == LogicalKeyboardKey.keyV;
-      final isModifierPressed = event.isMetaPressed || event.isControlPressed;
+      final isModifierPressed =
+          HardwareKeyboard.instance.isMetaPressed ||
+          HardwareKeyboard.instance.isControlPressed;
 
       if (isPaste && isModifierPressed) {
         handlePaste();
-        return;
+        return KeyEventResult.handled;
       }
 
       final enterToSend = ref.read(appSettingsNotifierProvider).enterToSend;
       final isEnter = event.logicalKey == LogicalKeyboardKey.enter;
 
       if (isEnter) {
-        if (enterToSend && !isModifierPressed) {
+        if (isModifierPressed) {
+          insertNewLine();
+          return KeyEventResult.handled;
+        } else if (enterToSend) {
           send();
-        } else if (!enterToSend && isModifierPressed) {
-          send();
-        }
+          return KeyEventResult.handled;
         }
       }
 
-    return Material(
-      elevation: 8,
-      color: Theme.of(context).colorScheme.surface,
+      return KeyEventResult.ignored;
+    };
+
+    return Container(
+      margin: const EdgeInsets.all(16),
+      child: Material(
+        elevation: 2,
+        color: Theme.of(context).colorScheme.surfaceContainerHighest,
+        borderRadius: BorderRadius.circular(32),
+        child: Padding(
+          padding: const EdgeInsets.symmetric(vertical: 6, horizontal: 8),
           child: Column(
             children: [
               if (attachments.isNotEmpty)
@@ -143,12 +156,20 @@ class ChatInput extends HookConsumerWidget {
                   messageForwardingTo != null ||
                   messageEditingTo != null)
                 Container(
-              padding: const EdgeInsets.symmetric(horizontal: 16),
-              decoration: BoxDecoration(
-                color: Theme.of(context).colorScheme.surfaceContainer,
-                borderRadius: BorderRadius.circular(8),
+                  padding: const EdgeInsets.symmetric(
+                    horizontal: 16,
+                    vertical: 4,
+                  ),
+                  decoration: BoxDecoration(
+                    color: Theme.of(context).colorScheme.surfaceContainerHigh,
+                    borderRadius: BorderRadius.circular(32),
+                  ),
+                  margin: const EdgeInsets.only(
+                    left: 8,
+                    right: 8,
+                    top: 8,
+                    bottom: 4,
                   ),
-              margin: const EdgeInsets.only(left: 8, right: 8, top: 8),
                   child: Row(
                     children: [
                       Icon(
@@ -173,20 +194,18 @@ class ChatInput extends HookConsumerWidget {
                           overflow: TextOverflow.ellipsis,
                         ),
                       ),
-                  IconButton(
-                    icon: const Icon(Icons.close, size: 20),
-                    onPressed: onClear,
-                    padding: EdgeInsets.zero,
-                    style: ButtonStyle(
-                      minimumSize: WidgetStatePropertyAll(Size(28, 28)),
+                      SizedBox(
+                        width: 28,
+                        height: 28,
+                        child: InkWell(
+                          onTap: onClear,
+                          child: const Icon(Icons.close, size: 20).center(),
                         ),
                       ),
                     ],
                   ),
                 ),
-          Padding(
-            padding: const EdgeInsets.symmetric(vertical: 6, horizontal: 8),
-            child: Row(
+              Row(
                 children: [
                   Row(
                     mainAxisSize: MainAxisSize.min,
@@ -260,32 +279,10 @@ class ChatInput extends HookConsumerWidget {
                     ],
                   ),
                   Expanded(
-                  child: RawKeyboardListener(
-                    focusNode: FocusNode(),
-                    onKey: (event) => handleKeyPress(context, ref, event),
                     child: TextField(
                       focusNode: inputFocusNode,
                       controller: messageController,
-                      onSubmitted:
-                          (enterToSend && isMobile)
-                              ? (_) {
-                                send();
-                              }
-                              : null,
-                      keyboardType:
-                          (enterToSend && isMobile)
-                              ? TextInputType.text
-                              : TextInputType.multiline,
-                      textInputAction: TextInputAction.send,
-                      inputFormatters: [
-                        if (enterToSend && !isMobile)
-                          TextInputFormatter.withFunction((oldValue, newValue) {
-                            if (newValue.text.endsWith('\n')) {
-                              return oldValue;
-                            }
-                            return newValue;
-                          }),
-                      ],
+                      keyboardType: TextInputType.multiline,
                       decoration: InputDecoration(
                         hintText:
                             (chatRoom.type == 1 && chatRoom.name == null)
@@ -314,17 +311,17 @@ class ChatInput extends HookConsumerWidget {
                           (_) => FocusManager.instance.primaryFocus?.unfocus(),
                     ),
                   ),
-                ),
                   IconButton(
                     icon: const Icon(Icons.send),
                     color: Theme.of(context).colorScheme.primary,
                     onPressed: send,
                   ),
                 ],
-            ).padding(bottom: MediaQuery.of(context).padding.bottom),
               ),
             ],
           ),
+        ),
+      ),
     );
   }
 }

lib/widgets/chat/message_content.dart

@@ -56,7 +56,7 @@ class MessageContent extends StatelessWidget {
       case 'messages.update.links':
         return Row(
           mainAxisSize: MainAxisSize.min,
-          crossAxisAlignment: CrossAxisAlignment.center,
+          crossAxisAlignment: CrossAxisAlignment.start,
           children: [
             Icon(
               Symbols.edit,
@@ -64,10 +64,11 @@ class MessageContent extends StatelessWidget {
               color: Theme.of(
                 context,
               ).colorScheme.onSurfaceVariant.withOpacity(0.6),
-            ),
+            ).padding(top: 2),
             const Gap(4),
             if (item.meta['previous_content'] is String)
-              PrettyDiffText(
+              Flexible(
+                child: PrettyDiffText(
                   oldText: item.meta['previous_content'],
                   newText: item.content ?? 'Edited a message',
                   defaultTextStyle: Theme.of(
@@ -86,6 +87,7 @@ class MessageContent extends StatelessWidget {
                       context,
                     ).colorScheme.onSurfaceVariant.withOpacity(0.7),
                   ),
+                ),
               )
             else
               Text(
@@ -104,11 +106,13 @@ class MessageContent extends StatelessWidget {
           mainAxisSize: MainAxisSize.min,
           crossAxisAlignment: CrossAxisAlignment.start,
           children: [
-            MarkdownTextContent(
+            Flexible(
+              child: MarkdownTextContent(
                 content: item.content ?? '*${item.type} has no content*',
                 isSelectable: true,
                 linesMargin: EdgeInsets.zero,
               ),
+            ),
             if (translatedText?.isNotEmpty ?? false)
               ...([
                 ConstrainedBox(