From 0db6870edb6b7829ca87f5e937268d11aed01c0c Mon Sep 17 00:00:00 2001
From: Danila Fedorin <danila.fedorin@gmail.com>
Date: Fri, 23 Jul 2021 15:52:29 -0700
Subject: [PATCH] Flesh out the fallback formatting a bit.

---
 .../room/timeline/entries/BaseEventEntry.js   | 46 +++++++++++++++----
 1 file changed, 38 insertions(+), 8 deletions(-)

diff --git a/src/matrix/room/timeline/entries/BaseEventEntry.js b/src/matrix/room/timeline/entries/BaseEventEntry.js
index 5c4ef6cf..f59d15f4 100644
--- a/src/matrix/room/timeline/entries/BaseEventEntry.js
+++ b/src/matrix/room/timeline/entries/BaseEventEntry.js
@@ -151,18 +151,48 @@ export class BaseEventEntry extends BaseEntry {
         return createAnnotation(this.id, key);
     }
 
-    _formatReplyBody() {
-        // This is just a rough sketch for now.
-        // TODO case-by-case formatting
+    _fallbackBlurb() {
+        switch (this.content.msgtype) {
+            case "m.file":
+                return "sent a file.";
+            case "m.image":
+                return "sent an image.";
+            case "m.video":
+                return "sent a video.";
+            case "m.audio":
+                return "sent an audio file.";
+        }
+    }
+
+    _fallbackPrefix() {
+        return this.content.msgtype === "m.emote" ? "* " : "";
+    }
+
+    _replyFormattedFallback() {
         // TODO check for absense?
-        const bodyLines = this.content.body.split("\n");
-        const sender = this.sender;
-        bodyLines[0] = `<${sender}> ${bodyLines[0]}`
-        return `> ${bodyLines.join("\n> ")}\n\n`;
+        // TODO escape unformatted body if needed
+        const body = this._fallbackBlurb() || this.content.formatted_body || this.content.body;
+        const prefix = this._fallbackPrefix();
+        return `<mx-reply>
+          <blockquote>
+            In reply to
+            ${prefix}<a href="https://matrix.to/#/${this.sender}">${this.displayName}</a>
+            <br />
+            ${body}
+          </blockquote>
+        </mx-reply>`
+    }
+
+    _replyBodyFallback() {
+        // TODO check for absense?
+        const body = this._fallbackBlurb() || this.content.body;
+        const bodyLines = body.split("\n");
+        bodyLines[0] = `> <${this.sender}> ${bodyLines[0]}`
+        return `${bodyLines.join("\n> ")}`;
     }
 
     reply(msgtype, body) {
-        return createReply(this.id, msgtype, this._formatReplyBody() + body);
+        return createReply(this.id, msgtype, this._replyBodyFallback() + '\n\n' + body);
     }
 
     /** takes both remote event id and local txn id into account, see overriding in PendingEventEntry */