Skip to content

Commit 1aa6a07

Browse files
gnpricechrisbobbe
authored andcommitted
msglist [nfc]: Have MessageWithSender take the whole message-item object
This will make things more convenient when we add another field on these which MessageWithSender will want to consult.
1 parent 11938c7 commit 1aa6a07

File tree

1 file changed

+5
-7
lines changed

1 file changed

+5
-7
lines changed

lib/widgets/message_list.dart

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import 'package:flutter/material.dart';
55
import 'package:intl/intl.dart';
66

77
import '../api/model/model.dart';
8-
import '../model/content.dart';
98
import '../model/message_list.dart';
109
import '../model/narrow.dart';
1110
import '../model/store.dart';
@@ -391,7 +390,7 @@ class MessageItem extends StatelessWidget {
391390
child: Column(children: [
392391
DecoratedBox(
393392
decoration: borderDecoration,
394-
child: MessageWithSender(message: message, content: item.content)),
393+
child: MessageWithSender(item: item)),
395394
if (trailing != null && item.isLastInBlock) trailing!,
396395
]));
397396

@@ -564,14 +563,13 @@ class RecipientHeaderChevronContainer extends StatelessWidget {
564563

565564
/// A Zulip message, showing the sender's name and avatar.
566565
class MessageWithSender extends StatelessWidget {
567-
const MessageWithSender(
568-
{super.key, required this.message, required this.content});
566+
const MessageWithSender({super.key, required this.item});
569567

570-
final Message message;
571-
final ZulipContent content;
568+
final MessageListMessageItem item;
572569

573570
@override
574571
Widget build(BuildContext context) {
572+
final message = item.message;
575573
final time = _kMessageTimestampFormat
576574
.format(DateTime.fromMillisecondsSinceEpoch(1000 * message.timestamp));
577575

@@ -601,7 +599,7 @@ class MessageWithSender extends StatelessWidget {
601599
child: Text(message.senderFullName, // TODO get from user data
602600
style: const TextStyle(fontWeight: FontWeight.bold))),
603601
const SizedBox(height: 4),
604-
MessageContent(message: message, content: content),
602+
MessageContent(message: message, content: item.content),
605603
])),
606604
Container(
607605
width: 80,

0 commit comments

Comments
 (0)