Timeline: Mark messages read sooner, and a little more often

Co-authored-by: Scott Nonnenberg <scott@signal.org>
signalapp · Apr 20, 2022 · 71fb4eb · 71fb4eb
1 parent a384d7b
commit 71fb4eb
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/ts/components/conversation/Timeline.tsx b/ts/components/conversation/Timeline.tsx
@@ -541,17 +541,13 @@ export class Timeline extends React.Component<
     this.intersectionObserver.observe(atBottomDetectorEl);
   }
 
-  private markNewestBottomVisibleMessageRead = throttle(
-    (): void => {
-      const { markMessageRead } = this.props;
-      const { newestBottomVisibleMessageId } = this.state;
-      if (newestBottomVisibleMessageId) {
-        markMessageRead(newestBottomVisibleMessageId);
-      }
-    },
-    500,
-    { leading: false }
-  );
+  private markNewestBottomVisibleMessageRead = throttle((): void => {
+    const { markMessageRead } = this.props;
+    const { newestBottomVisibleMessageId } = this.state;
+    if (newestBottomVisibleMessageId) {
+      markMessageRead(newestBottomVisibleMessageId);
+    }
+  }, 500);
 
   public override componentDidMount(): void {
     const containerEl = this.containerRef.current;
@@ -674,6 +670,8 @@ export class Timeline extends React.Component<
           numberToKeepAtBottom: this.maxVisibleRows,
         });
       }
+    } else {
+      this.markNewestBottomVisibleMessageRead();
     }
   }