Merge pull request #591 from matrix-org/luke/fix-wantsdateseparator-performance

Fix performance issues with wantsDateSeperator
This commit is contained in:
Richard van der Hoff 2016-12-15 12:45:18 +00:00 committed by GitHub
commit 238a30789f

View file

@ -21,6 +21,8 @@ var sdk = require('../../index');
var MatrixClientPeg = require('../../MatrixClientPeg')
const MILLIS_IN_DAY = 86400000;
/* (almost) stateless UI component which builds the event tiles in the room timeline.
*/
module.exports = React.createClass({
@ -466,9 +468,7 @@ module.exports = React.createClass({
// here.
return !this.props.suppressFirstDateSeparator;
}
return (new Date(prevEvent.getTs()).toDateString()
!== new Date(nextEventTs).toDateString());
return Math.floor(prevEvent.getTs() / MILLIS_IN_DAY) !== Math.floor(nextEventTs / MILLIS_IN_DAY);
},
// get a list of read receipts that should be shown next to this event