msglist: Remove 11px bottom padding in MessageItem at end of feed

See discussions:
  #1453 (comment)
  https://chat.zulip.org/#narrow/channel/48-mobile/topic/space.20at.20end.20of.20thread/near/2203417

Author: chrisbobbe

lib/model/message_list.dart

@@ -44,10 +44,12 @@ sealed class MessageListMessageBaseItem extends MessageListItem {
   ZulipMessageContent get content;
   bool showSender;
   bool isLastInBlock;
+  bool isLastInFeed;
 
   MessageListMessageBaseItem({
     required this.showSender,
     required this.isLastInBlock,
+    required this.isLastInFeed,
   });
 }
 
@@ -63,6 +65,7 @@ class MessageListMessageItem extends MessageListMessageBaseItem {
     this.content, {
     required super.showSender,
     required super.isLastInBlock,
+    required super.isLastInFeed,
   });
 }
 
@@ -77,6 +80,7 @@ class MessageListOutboxMessageItem extends MessageListMessageBaseItem {
     this.message, {
     required super.showSender,
     required super.isLastInBlock,
+    required super.isLastInFeed,
   }) : content = ZulipContent(nodes: [
     ParagraphNode(links: null, nodes: [TextNode(message.contentMarkdown)]),
   ]);
@@ -443,6 +447,9 @@ mixin _MessageSequence {
       final prevMessageItem = items.last as MessageListMessageBaseItem;
       assert(identical(prevMessageItem.message, prevMessage));
       assert(prevMessageItem.isLastInBlock);
+      assert(prevMessageItem.isLastInFeed);
+
+      prevMessageItem.isLastInFeed = false;
 
       if (!haveSameRecipient(prevMessage, message)) {
         items.add(MessageListRecipientHeaderItem(message));
@@ -462,6 +469,7 @@ mixin _MessageSequence {
     assert(identical(item.message, message));
     assert(item.showSender == !canShareSender);
     assert(item.isLastInBlock);
+    assert(item.isLastInFeed);
     if (shouldSetMiddleItem) {
       middleItem = items.length;
     }
@@ -482,7 +490,7 @@ mixin _MessageSequence {
       shouldSetMiddleItem: index == middleMessage,
       prevMessage: prevMessage,
       buildItem: (bool canShareSender) => MessageListMessageItem(
-        message, content, showSender: !canShareSender, isLastInBlock: true));
+        message, content, showSender: !canShareSender, isLastInBlock: true, isLastInFeed: true));
   }
 
   /// Append to [items] based on the index-th message in [outboxMessages].
@@ -500,7 +508,7 @@ mixin _MessageSequence {
       shouldSetMiddleItem: index == 0 && middleMessage == messages.length,
       prevMessage: prevMessage,
       buildItem: (bool canShareSender) => MessageListOutboxMessageItem(
-        message, showSender: !canShareSender, isLastInBlock: true));
+        message, showSender: !canShareSender, isLastInBlock: true, isLastInFeed: true));
   }
 
   /// Remove items associated with [outboxMessages] from [items].
@@ -519,6 +527,7 @@ mixin _MessageSequence {
     if (items.isNotEmpty) {
       final lastItem = items.last as MessageListMessageItem;
       lastItem.isLastInBlock = true;
+      lastItem.isLastInFeed = true;
     }
     if (middleMessage == messages.length) middleItem = items.length;
   }

lib/widgets/message_list.dart

@@ -1334,9 +1334,7 @@ class MessageItem extends StatelessWidget {
           MessageListMessageItem() => MessageWithPossibleSender(item: item),
           MessageListOutboxMessageItem() => OutboxMessageWithPossibleSender(item: item),
         },
-        // TODO refine this padding; discussion:
-        //   https://github.com/zulip/zulip-flutter/pull/1453#discussion_r2106526985
-        if (item.isLastInBlock) const SizedBox(height: 11),
+        if (item.isLastInBlock && !item.isLastInFeed) const SizedBox(height: 11),
       ]));
     if (item case MessageListMessageItem(:final message)) {
       child = _UnreadMarker(

test/model/message_list_test.dart

@@ -3037,7 +3037,8 @@ void checkInvariants(MessageListView model) {
           || MessageListOutboxMessageItem()
           || MessageListDateSeparatorItem() => false,
           MessageListRecipientHeaderItem()  => true,
-        });
+        })
+      ..isLastInFeed.equals(i == model.items.length);
   }
   check(model.items).length.equals(i);
 
@@ -3082,6 +3083,7 @@ extension MessageListMessageBaseItemChecks on Subject<MessageListMessageBaseItem
   Subject<ZulipMessageContent> get content => has((x) => x.content, 'content');
   Subject<bool> get showSender => has((x) => x.showSender, 'showSender');
   Subject<bool> get isLastInBlock => has((x) => x.isLastInBlock, 'isLastInBlock');
+  Subject<bool> get isLastInFeed => has((x) => x.isLastInFeed, 'isLastInFeed');
 }
 
 extension MessageListMessageItemChecks on Subject<MessageListMessageItem> {