From c94715150d96ad1961684743c04157047e713005 Mon Sep 17 00:00:00 2001 From: Bruno Windels Date: Wed, 17 Jul 2019 10:18:51 +0200 Subject: [PATCH] as both RM and RR move past invisible events, remove the flag --- src/components/structures/TimelinePanel.js | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/components/structures/TimelinePanel.js b/src/components/structures/TimelinePanel.js index 7f18589ba0..e367544492 100644 --- a/src/components/structures/TimelinePanel.js +++ b/src/components/structures/TimelinePanel.js @@ -658,7 +658,6 @@ const TimelinePanel = React.createClass({ const lastReadEventIndex = this._getLastDisplayedEventIndex({ ignoreOwn: true, - allowEventsWithoutTiles: true, }); if (lastReadEventIndex === null) { shouldSendRR = false; @@ -747,7 +746,6 @@ const TimelinePanel = React.createClass({ const lastDisplayedIndex = this._getLastDisplayedEventIndex({ allowPartial: true, ignoreEchoes: true, - allowEventsWithoutTiles: true, }); if (lastDisplayedIndex === null) { @@ -1132,7 +1130,6 @@ const TimelinePanel = React.createClass({ const ignoreOwn = opts.ignoreOwn || false; const ignoreEchoes = opts.ignoreEchoes || false; const allowPartial = opts.allowPartial || false; - const allowEventsWithoutTiles = opts.allowEventsWithoutTiles || false; const messagePanel = this.refs.messagePanel; if (messagePanel === undefined) return null; @@ -1153,8 +1150,7 @@ const TimelinePanel = React.createClass({ return false; }; - // if allowEventsWithoutTiles is enabled, we keep track - // of how many of the adjacent events didn't have a tile + // We keep track of how many of the adjacent events didn't have a tile // but should have the read receipt moved past them, so // we can include those once we find the last displayed (visible) event. // The counter is not started for events we don't want @@ -1183,7 +1179,7 @@ const TimelinePanel = React.createClass({ (ignoreOwn && ev.sender && ev.sender.userId == myUserId); // own message const isWithoutTile = !EventTile.haveTileForEvent(ev) || shouldHideEvent(ev); - if (allowEventsWithoutTiles && (isWithoutTile || !node)) { + if (isWithoutTile || !node) { // don't start counting if the event should be ignored, // but continue counting if we were already so the offset // to the previous invisble event that didn't need to be ignored