msglist: Adjust padding around message list sender

Paddings could be combined (for example the padding around the user avatar and name could be included with the outer padding around the `senderWidget`) but left according to the Figma specifications as this increases the hit target of the sender so is useful to have.
sirpengi · Dec 8, 2023 · c86f1fb · c86f1fb
1 parent db1401d
commit c86f1fb
Showing 1 changed file with 26 additions and 27 deletions.
diff --git a/lib/widgets/message_list.dart b/lib/widgets/message_list.dart
@@ -785,19 +785,20 @@ class MessageWithPossibleSender extends StatelessWidget {
 
     final Widget senderWidget;
     if (item.showSender) {
-      senderWidget = Column(
-        children: [
-          const SizedBox(height: 3),
-          Row(
-            mainAxisAlignment: MainAxisAlignment.spaceBetween,
-            crossAxisAlignment: CrossAxisAlignment.baseline,
-            textBaseline: TextBaseline.alphabetic,
-            children: [
-              Flexible(
-                child: GestureDetector(
-                  onTap: () => Navigator.push(context,
-                    ProfilePage.buildRoute(context: context,
-                      userId: message.senderId)),
+      senderWidget = Padding(
+        padding: const EdgeInsets.symmetric(vertical: 2),
+        child: Row(
+          mainAxisAlignment: MainAxisAlignment.spaceBetween,
+          crossAxisAlignment: CrossAxisAlignment.baseline,
+          textBaseline: TextBaseline.alphabetic,
+          children: [
+            Flexible(
+              child: GestureDetector(
+                onTap: () => Navigator.push(context,
+                  ProfilePage.buildRoute(context: context,
+                    userId: message.senderId)),
+                child: Padding(
+                  padding: const EdgeInsets.symmetric(vertical: 2),
                   child: Row(
                     children: [
                       Avatar(size: 32, borderRadius: 3, userId: message.senderId),
@@ -812,20 +813,18 @@ class MessageWithPossibleSender extends StatelessWidget {
                               wghtIfPlatformRequestsBold: 900)),
                           overflow: TextOverflow.ellipsis),
                       ),
-                    ])),
-              ),
-              Text(time,
-                style: TextStyle(
-                  color: _kMessageTimestampColor,
-                  fontFamily: 'Source Sans 3',
-                  fontSize: 16,
-                  height: (18 / 16),
-                  fontFeatures: const [FontFeature.enable('c2sc'), FontFeature.enable('smcp')],
-                ).merge(weightVariableTextStyle(context))),
-            ]),
-          const SizedBox(height: 4),
-        ],
-      );
+                    ]),
+                )),
+            ),
+            Text(time,
+              style: TextStyle(
+                color: _kMessageTimestampColor,
+                fontFamily: 'Source Sans 3',
+                fontSize: 16,
+                height: (18 / 16),
+                fontFeatures: const [FontFeature.enable('c2sc'), FontFeature.enable('smcp')],
+              ).merge(weightVariableTextStyle(context))),
+          ]));
     } else {
       senderWidget = const SizedBox.shrink();
     }