De-duplicate notif badge code
We had two different places we were deciding whether to show a badge. Let's just have one.
This commit is contained in:
parent
9591e6b0d3
commit
c57d93702a
2 changed files with 8 additions and 16 deletions
|
@ -26,12 +26,12 @@ export const MUTE = 'mute';
|
||||||
export const BADGE_STATES = [ALL_MESSAGES, ALL_MESSAGES_LOUD];
|
export const BADGE_STATES = [ALL_MESSAGES, ALL_MESSAGES_LOUD];
|
||||||
export const MENTION_BADGE_STATES = [...BADGE_STATES, MENTIONS_ONLY];
|
export const MENTION_BADGE_STATES = [...BADGE_STATES, MENTIONS_ONLY];
|
||||||
|
|
||||||
function _shouldShowNotifBadge(roomNotifState) {
|
export function shouldShowNotifBadge(roomNotifState) {
|
||||||
const showBadgeInStates = [ALL_MESSAGES, ALL_MESSAGES_LOUD];
|
const showBadgeInStates = [ALL_MESSAGES, ALL_MESSAGES_LOUD];
|
||||||
return showBadgeInStates.indexOf(roomNotifState) > -1;
|
return showBadgeInStates.indexOf(roomNotifState) > -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
function _shouldShowMentionBadge(roomNotifState) {
|
export function shouldShowMentionBadge(roomNotifState) {
|
||||||
return roomNotifState !== MUTE;
|
return roomNotifState !== MUTE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -41,8 +41,8 @@ export function aggregateNotificationCount(rooms) {
|
||||||
const highlight = room.getUnreadNotificationCount('highlight') > 0;
|
const highlight = room.getUnreadNotificationCount('highlight') > 0;
|
||||||
const notificationCount = room.getUnreadNotificationCount();
|
const notificationCount = room.getUnreadNotificationCount();
|
||||||
|
|
||||||
const notifBadges = notificationCount > 0 && _shouldShowNotifBadge(roomNotifState);
|
const notifBadges = notificationCount > 0 && shouldShowNotifBadge(roomNotifState);
|
||||||
const mentionBadges = highlight && _shouldShowMentionBadge(roomNotifState);
|
const mentionBadges = highlight && shouldShowMentionBadge(roomNotifState);
|
||||||
const badges = notifBadges || mentionBadges;
|
const badges = notifBadges || mentionBadges;
|
||||||
|
|
||||||
if (badges) {
|
if (badges) {
|
||||||
|
@ -60,8 +60,8 @@ export function getRoomHasBadge(room) {
|
||||||
const highlight = room.getUnreadNotificationCount('highlight') > 0;
|
const highlight = room.getUnreadNotificationCount('highlight') > 0;
|
||||||
const notificationCount = room.getUnreadNotificationCount();
|
const notificationCount = room.getUnreadNotificationCount();
|
||||||
|
|
||||||
const notifBadges = notificationCount > 0 && _shouldShowNotifBadge(roomNotifState);
|
const notifBadges = notificationCount > 0 && shouldShowNotifBadge(roomNotifState);
|
||||||
const mentionBadges = highlight && _shouldShowMentionBadge(roomNotifState);
|
const mentionBadges = highlight && shouldShowMentionBadge(roomNotifState);
|
||||||
|
|
||||||
return notifBadges || mentionBadges;
|
return notifBadges || mentionBadges;
|
||||||
}
|
}
|
||||||
|
|
|
@ -67,14 +67,6 @@ module.exports = React.createClass({
|
||||||
});
|
});
|
||||||
},
|
},
|
||||||
|
|
||||||
_shouldShowNotifBadge: function() {
|
|
||||||
return RoomNotifs.BADGE_STATES.includes(this.state.notifState);
|
|
||||||
},
|
|
||||||
|
|
||||||
_shouldShowMentionBadge: function() {
|
|
||||||
return RoomNotifs.MENTION_BADGE_STATES.includes(this.state.notifState);
|
|
||||||
},
|
|
||||||
|
|
||||||
_isDirectMessageRoom: function(roomId) {
|
_isDirectMessageRoom: function(roomId) {
|
||||||
const dmRooms = DMRoomMap.shared().getUserIdForRoomId(roomId);
|
const dmRooms = DMRoomMap.shared().getUserIdForRoomId(roomId);
|
||||||
return Boolean(dmRooms);
|
return Boolean(dmRooms);
|
||||||
|
@ -301,8 +293,8 @@ module.exports = React.createClass({
|
||||||
const notificationCount = this.props.notificationCount;
|
const notificationCount = this.props.notificationCount;
|
||||||
// var highlightCount = this.props.room.getUnreadNotificationCount("highlight");
|
// var highlightCount = this.props.room.getUnreadNotificationCount("highlight");
|
||||||
|
|
||||||
const notifBadges = notificationCount > 0 && this._shouldShowNotifBadge();
|
const notifBadges = notificationCount > 0 && RoomNotifs.shouldShowNotifBadge(this.state.notifState);
|
||||||
const mentionBadges = this.props.highlight && this._shouldShowMentionBadge();
|
const mentionBadges = this.props.highlight && RoomNotifs.shouldShowMentionBadge(this.state.notifState);
|
||||||
const badges = notifBadges || mentionBadges;
|
const badges = notifBadges || mentionBadges;
|
||||||
|
|
||||||
let subtext = null;
|
let subtext = null;
|
||||||
|
|
Loading…
Reference in a new issue