From fb6beb4acabeb04464f27972f232468098883b65 Mon Sep 17 00:00:00 2001 From: DatGuy1 Date: Tue, 26 Mar 2024 20:51:16 +0200 Subject: [PATCH] fix: add trailing space if deletion link included in image upload response (#5269) Co-authored-by: Rasmus Karlsson --- CHANGELOG.md | 1 + src/messages/MessageBuilder.cpp | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 1c039db91..5245db53b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -113,6 +113,7 @@ - Bugfix: Fixed split header tooltips appearing too tall. (#5232) - Bugfix: Fixed past messages not showing in the search popup after adding a channel. (#5248) - Bugfix: Detect when OBS is running on MacOS. (#5260) +- Bugfix: Fixed a missing space when the image uploader provided a delete link. (#5269) - Bugfix: Remove ":" from the message the user is replying to if it's a /me message. (#5263) - Dev: Run miniaudio in a separate thread, and simplify it to not manage the device ourselves. There's a chance the simplification is a bad idea. (#4978) - Dev: Change clang-format from v14 to v16. (#4929) diff --git a/src/messages/MessageBuilder.cpp b/src/messages/MessageBuilder.cpp index af5dadcf7..cc71558f6 100644 --- a/src/messages/MessageBuilder.cpp +++ b/src/messages/MessageBuilder.cpp @@ -543,7 +543,7 @@ MessageBuilder::MessageBuilder(ImageUploaderResultTag /*unused*/, // This also ensures that the LinkResolver doesn't get these links. addText(imageLink, MessageColor::Link) ->setLink({Link::Url, imageLink}) - ->setTrailingSpace(false); + ->setTrailingSpace(!deletionLink.isEmpty()); if (!deletionLink.isEmpty()) {