From f9cf98438163eb2380906033d599ad1554bda1a2 Mon Sep 17 00:00:00 2001
From: Tyler Brown Cifu Shuster <tyler@fivepaths.com>
Date: Tue, 8 Sep 2020 15:20:40 -0700
Subject: [PATCH] chat: prevent message rerender

---
 .../apps/chat/components/lib/chat-message.tsx | 165 +++++++++---------
 1 file changed, 83 insertions(+), 82 deletions(-)

diff --git a/pkg/interface/src/views/apps/chat/components/lib/chat-message.tsx b/pkg/interface/src/views/apps/chat/components/lib/chat-message.tsx
index 12174803a..e29e18802 100644
--- a/pkg/interface/src/views/apps/chat/components/lib/chat-message.tsx
+++ b/pkg/interface/src/views/apps/chat/components/lib/chat-message.tsx
@@ -5,87 +5,88 @@ import { Message } from "./message";
 import { Envelope } from "~/types/chat-update";
 import _ from "lodash";
 
-export const ChatMessage = (props) => {
-  const {
-    msg,
-    previousMsg,
-    nextMsg,
-    isFirstUnread,
-    group,
-    association,
-    contacts,
-    unreadRef,
-    hideAvatars,
-    hideNicknames,
-    remoteContentPolicy,
-    className = ''
-  } = props;
-
-  // Render sigil if previous message is not by the same sender
-  const aut = ["author"];
-  const renderSigil =
-    _.get(nextMsg, aut) !== _.get(msg, aut, msg.author);
-  const paddingTop = renderSigil;
-  const paddingBot =
-    _.get(previousMsg, aut) !== _.get(msg, aut, msg.author);
-
-  const when = ["when"];
-  const dayBreak =
-    moment(_.get(nextMsg, when)).format("YYYY.MM.DD") !==
-    moment(_.get(msg, when)).format("YYYY.MM.DD");
-
-  const messageElem = (
-    <Message
-      key={msg.uid}
-      msg={msg}
-      renderSigil={renderSigil}
-      paddingTop={paddingTop}
-      paddingBot={paddingBot}
-      pending={Boolean(msg.pending)}
-      group={group}
-      contacts={contacts}
-      association={association}
-      hideNicknames={hideNicknames}
-      hideAvatars={hideAvatars}
-      remoteContentPolicy={remoteContentPolicy}
-      className={className}
-    />
-  );
-
-  if (isFirstUnread) {
-    return (
-      <Fragment key={msg.uid}>
-        {messageElem}
-        <div ref={unreadRef}
-             className="mv2 green2 flex items-center f9">
-          <hr className="dn-s ma0 w2 b--green2 bt-0" />
-          <p className="mh4">New messages below</p>
-          <hr className="ma0 flex-grow-1 b--green2 bt-0" />
-          {dayBreak && (
-            <p className="gray2 mh4">
-              {moment(_.get(msg, when)).calendar()}
-            </p>
-          )}
-          <hr
-            style={{ width: "calc(50% - 48px)" }}
-            className="b--green2 ma0 bt-0"
-          />
-        </div>
-      </Fragment>
+export class ChatMessage extends PureComponent {
+  render() {
+    const {
+      msg,
+      previousMsg,
+      nextMsg,
+      isFirstUnread,
+      group,
+      association,
+      contacts,
+      unreadRef,
+      hideAvatars,
+      hideNicknames,
+      remoteContentPolicy,
+      className = ''
+    } = this.props;
+  
+    // Render sigil if previous message is not by the same sender
+    const aut = ["author"];
+    const renderSigil =
+      _.get(nextMsg, aut) !== _.get(msg, aut, msg.author);
+    const paddingTop = renderSigil;
+    const paddingBot =
+      _.get(previousMsg, aut) !== _.get(msg, aut, msg.author);
+  
+    const when = ["when"];
+    const dayBreak =
+      moment(_.get(nextMsg, when)).format("YYYY.MM.DD") !==
+      moment(_.get(msg, when)).format("YYYY.MM.DD");
+  
+    const messageElem = (
+      <Message
+        key={msg.uid}
+        msg={msg}
+        renderSigil={renderSigil}
+        paddingTop={paddingTop}
+        paddingBot={paddingBot}
+        pending={Boolean(msg.pending)}
+        group={group}
+        contacts={contacts}
+        association={association}
+        hideNicknames={hideNicknames}
+        hideAvatars={hideAvatars}
+        remoteContentPolicy={remoteContentPolicy}
+        className={className}
+      />
     );
-  } else if (dayBreak) {
-    return (
-      <Fragment key={msg.uid}>
-        <div
-          className="pv3 gray2 b--gray2 flex items-center justify-center f9 "
-        >
-          <p>{moment(_.get(msg, when)).calendar()}</p>
-        </div>
-        {messageElem}
-      </Fragment>
-    );
-  } else {
-    return messageElem;
+  
+    if (isFirstUnread) {
+      return (
+        <Fragment key={msg.uid}>
+          {messageElem}
+          <div ref={unreadRef}
+               className="mv2 green2 flex items-center f9">
+            <hr className="dn-s ma0 w2 b--green2 bt-0" />
+            <p className="mh4">New messages below</p>
+            <hr className="ma0 flex-grow-1 b--green2 bt-0" />
+            {dayBreak && (
+              <p className="gray2 mh4">
+                {moment(_.get(msg, when)).calendar()}
+              </p>
+            )}
+            <hr
+              style={{ width: "calc(50% - 48px)" }}
+              className="b--green2 ma0 bt-0"
+            />
+          </div>
+        </Fragment>
+      );
+    } else if (dayBreak) {
+      return (
+        <Fragment key={msg.uid}>
+          <div
+            className="pv3 gray2 b--gray2 flex items-center justify-center f9 "
+          >
+            <p>{moment(_.get(msg, when)).calendar()}</p>
+          </div>
+          {messageElem}
+        </Fragment>
+      );
+    } else {
+      return messageElem;
+    }
   }
-};
-
+}
\ No newline at end of file