We can have numUnreadMentions > 0 and numUnreadMessages == 0, in this case, display both icons.
This commit is contained in:
parent
5339acfae0
commit
3a7804b3b0
2 changed files with 14 additions and 10 deletions
|
|
@ -204,8 +204,8 @@ private fun RowScope.NotificationIcons(room: RoomListRoomSummary) {
|
|||
imageVector = CompoundIcons.Mention,
|
||||
tint = ElementTheme.colors.unreadIndicator,
|
||||
)
|
||||
}
|
||||
if (room.numUnreadMessages > 0) {
|
||||
UnreadIndicatorAtom()
|
||||
} else if (room.numUnreadMessages > 0) {
|
||||
UnreadIndicatorAtom()
|
||||
}
|
||||
}
|
||||
|
|
@ -217,13 +217,9 @@ private fun RowScope.NotificationIcons(room: RoomListRoomSummary) {
|
|||
imageVector = CompoundIcons.Mention,
|
||||
tint = ElementTheme.colors.unreadIndicator,
|
||||
)
|
||||
if (room.numUnreadMessages > 0) {
|
||||
UnreadIndicatorAtom()
|
||||
}
|
||||
} else {
|
||||
if (room.numUnreadMessages > 0) {
|
||||
UnreadIndicatorAtom(color = ElementTheme.colors.iconQuaternary)
|
||||
}
|
||||
UnreadIndicatorAtom()
|
||||
} else if (room.numUnreadMessages > 0) {
|
||||
UnreadIndicatorAtom(color = ElementTheme.colors.iconQuaternary)
|
||||
}
|
||||
}
|
||||
RoomNotificationMode.MUTE -> {
|
||||
|
|
@ -233,7 +229,7 @@ private fun RowScope.NotificationIcons(room: RoomListRoomSummary) {
|
|||
imageVector = CompoundIcons.NotificationsSolidOff,
|
||||
tint = ElementTheme.colors.iconQuaternary,
|
||||
)
|
||||
if (room.numUnreadMessages > 0) {
|
||||
if (room.numUnreadMessages > 0 || room.numUnreadMentions > 0) {
|
||||
UnreadIndicatorAtom(color = ElementTheme.colors.iconQuaternary)
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -69,6 +69,14 @@ open class RoomListRoomSummaryProvider : PreviewParameterProvider<RoomListRoomSu
|
|||
numUnreadMentions = 1,
|
||||
hasOngoingCall = hasCall,
|
||||
),
|
||||
aRoomListRoomSummary(
|
||||
name = roomNotificationMode.name,
|
||||
lastMessage = "New mentions" + if (hasCall) ", call" else "",
|
||||
notificationMode = roomNotificationMode,
|
||||
numUnreadMessages = 0,
|
||||
numUnreadMentions = 1,
|
||||
hasOngoingCall = hasCall,
|
||||
),
|
||||
)
|
||||
}.flatten()
|
||||
}.flatten(),
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue