Skip to content

Commit f1c6b56

Browse files
committed
msglist [nfc]: Fix a comment that wasn't quite right for starred/mentions
It looks like this comment was written for CombinedFeedNarrow and not updated when the other two cases were added. (Unlike with CombinedFeedNarrow, it's possible that some of `messageIds` weren't starred or mentioned in the first place, so it was inaccurate to say they "were" in the narrow.)
1 parent 8d68939 commit f1c6b56

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

lib/model/message_list.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1131,7 +1131,7 @@ class MessageListView with ChangeNotifier, _MessageSequence {
11311131
case CombinedFeedNarrow():
11321132
case MentionsNarrow():
11331133
case StarredMessagesNarrow():
1134-
// The messages were and remain in this narrow.
1134+
// The messages didn't enter or leave this narrow.
11351135
// TODO(#421): … except they may have become muted or not.
11361136
// We'll handle that at the same time as we handle muting itself changing.
11371137
// Recipient headers, and downstream of those, may change, though.

0 commit comments

Comments
 (0)