shouldCurrentMessageHideMetadata: Hide more often for less jumpiness
This commit is contained in:
parent
16f0f4262a
commit
97b03d8803
2 changed files with 3 additions and 8 deletions
|
@ -217,7 +217,7 @@ describe('<Timeline> utilities', () => {
|
||||||
assert.isTrue(shouldCurrentMessageHideMetadata(true, current, newer));
|
assert.isTrue(shouldCurrentMessageHideMetadata(true, current, newer));
|
||||||
});
|
});
|
||||||
|
|
||||||
it('returns true if all three have delivered status or above', () => {
|
it('returns true if both have delivered status or above', () => {
|
||||||
assert.isTrue(
|
assert.isTrue(
|
||||||
shouldCurrentMessageHideMetadata(true, defaultCurrent, defaultNewer)
|
shouldCurrentMessageHideMetadata(true, defaultCurrent, defaultNewer)
|
||||||
);
|
);
|
||||||
|
|
|
@ -88,18 +88,13 @@ export function shouldCurrentMessageHideMetadata(
|
||||||
case 'sending':
|
case 'sending':
|
||||||
return false;
|
return false;
|
||||||
case 'sent':
|
case 'sent':
|
||||||
return newerMessage.status === 'sent';
|
|
||||||
case 'delivered':
|
case 'delivered':
|
||||||
case 'read':
|
case 'read':
|
||||||
case 'viewed':
|
case 'viewed':
|
||||||
return (
|
return true;
|
||||||
newerMessage.status === 'delivered' ||
|
|
||||||
newerMessage.status === 'read' ||
|
|
||||||
newerMessage.status === 'viewed'
|
|
||||||
);
|
|
||||||
default:
|
default:
|
||||||
log.error(missingCaseError(message.status));
|
log.error(missingCaseError(message.status));
|
||||||
return false;
|
return true;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue