Change position of "Jump to ..." action for better consistency (#279)

This will better align with how a normal moderator would respond to the log: Before: "see log" -> "jump to message without knowing what changed" -> "read message diff" After: "see log" -> "read message diff" -> "jump to message for context" In addition, the change improves consistency with how reminders are shown. --------- Signed-off-by: Octol1ttle <[email protected]>
TeamOctolings · Mar 20, 2024 · 5cc04e9 · 5cc04e9
1 parent 0a930dc
commit 5cc04e9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/src/Responders/MessageDeletedResponder.cs b/src/Responders/MessageDeletedResponder.cs
@@ -83,10 +83,11 @@ public async Task<Result> RespondAsync(IMessageDelete gatewayEvent, Cancellation
 
         Messages.Culture = GuildSettings.Language.Get(cfg);
 
-        var builder = new StringBuilder().AppendLine(
-                string.Format(Messages.DescriptionActionJumpToChannel,
-                    Mention.Channel(gatewayEvent.ChannelID)))
-            .AppendLine(message.Content.InBlockCode());
+        var builder = new StringBuilder()
+            .AppendLine(message.Content.InBlockCode())
+            .AppendLine(
+                string.Format(Messages.DescriptionActionJumpToChannel, Mention.Channel(gatewayEvent.ChannelID))
+            );
 
         var embed = new EmbedBuilder()
             .WithSmallTitle(

diff --git a/src/Responders/MessageEditedResponder.cs b/src/Responders/MessageEditedResponder.cs
@@ -101,10 +101,11 @@ public async Task<Result> RespondAsync(IMessageUpdate gatewayEvent, Cancellation
 
         Messages.Culture = GuildSettings.Language.Get(cfg);
 
-        var builder = new StringBuilder().AppendLine(
-                string.Format(Messages.DescriptionActionJumpToMessage,
-                    $"https://discord.com/channels/{guildId}/{channelId}/{messageId}"))
-            .AppendLine(diff.AsMarkdown());
+        var builder = new StringBuilder()
+            .AppendLine(diff.AsMarkdown())
+            .AppendLine(string.Format(Messages.DescriptionActionJumpToMessage,
+                $"https://discord.com/channels/{guildId}/{channelId}/{messageId}")
+            );
 
         var embed = new EmbedBuilder()
             .WithSmallTitle(string.Format(Messages.CachedMessageEdited, message.Author.GetTag()), message.Author)