Fix environment workaround for translated text (#1411)

Author: laevandus

CHANGELOG.md

@@ -48,6 +48,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 - Remove `InjectedChannelInfo` from `ChatChannelListItemView` [#1338](https://github.com/GetStream/stream-chat-swiftui/pull/1338)
 - Rename empty state views from `No` prefix to `Empty` prefix [#1345](https://github.com/GetStream/stream-chat-swiftui/pull/1345)
 - Migrate all the old color tokens to new color tokens [#1350](https://github.com/GetStream/stream-chat-swiftui/pull/1350)
+- Replace `LinkDetectionTextView` with `StreamTextView` that uses `ChatMessage.attributedTextContent(layoutDirection:translationLanguage:)` [#1411](https://github.com/GetStream/stream-chat-swiftui/pull/1411)
 
 # [4.99.1](https://github.com/GetStream/stream-chat-swiftui/releases/tag/4.99.1)
 _April 01, 2026_

Sources/StreamChatSwiftUI/ChatMessageList/AttachmentTextView.swift

@@ -11,23 +11,29 @@ public struct AttachmentTextView<Factory: ViewFactory>: View {
     var factory: Factory
     var message: ChatMessage
     var availableWidth: CGFloat
+    var translationLanguage: TranslationLanguage?
 
     public init(
         factory: Factory = DefaultViewFactory.shared,
         message: ChatMessage,
-        availableWidth: CGFloat
+        availableWidth: CGFloat,
+        translationLanguage: TranslationLanguage? = nil
     ) {
         self.factory = factory
         self.message = message
         self.availableWidth = availableWidth
+        self.translationLanguage = translationLanguage
     }
 
     public var body: some View {
-        factory.makeStreamTextView(options: .init(message: message))
-            .padding(.horizontal, tokens.spacingXxs)
-            .fixedSize(horizontal: false, vertical: true)
-            .frame(maxWidth: maxTextWidth, alignment: .leading)
-            .accessibilityIdentifier("MessageTextView")
+        factory.makeStreamTextView(options: .init(
+            message: message,
+            translationLanguage: translationLanguage
+        ))
+        .padding(.horizontal, tokens.spacingXxs)
+        .fixedSize(horizontal: false, vertical: true)
+        .frame(maxWidth: maxTextWidth, alignment: .leading)
+        .accessibilityIdentifier("MessageTextView")
     }
 
     /// Limit text width for messages with portrait image attachment.

Sources/StreamChatSwiftUI/ChatMessageList/MessageAttachmentsView.swift

@@ -17,19 +17,22 @@ public struct MessageAttachmentsView<Factory: ViewFactory>: View {
     let message: ChatMessage
     let width: CGFloat
     let isFirst: Bool
+    let translationLanguage: TranslationLanguage?
     @Binding var scrolledId: String?
 
     public init(
         factory: Factory,
         message: ChatMessage,
         width: CGFloat,
         isFirst: Bool,
-        scrolledId: Binding<String?>
+        scrolledId: Binding<String?>,
+        translationLanguage: TranslationLanguage? = nil
     ) {
         self.factory = factory
         self.message = message
         self.width = width
         self.isFirst = isFirst
+        self.translationLanguage = translationLanguage
         self._scrolledId = scrolledId
     }
 
@@ -114,7 +117,8 @@ public struct MessageAttachmentsView<Factory: ViewFactory>: View {
                 factory.makeAttachmentTextView(
                     options: AttachmentTextViewOptions(
                         message: message,
-                        availableWidth: width
+                        availableWidth: width,
+                        translationLanguage: translationLanguage
                     )
                 )
             }

Sources/StreamChatSwiftUI/ChatMessageList/MessageContainerView.swift

@@ -110,7 +110,8 @@ struct MessageContainerView<Factory: ViewFactory>: View {
                         message: message,
                         contentWidth: contentWidth,
                         isFirst: showsAllInfo,
-                        scrolledId: $scrolledId
+                        scrolledId: $scrolledId,
+                        translationLanguage: messageViewModel.translationLanguage
                     )
                 }
             } else {
@@ -119,7 +120,8 @@ struct MessageContainerView<Factory: ViewFactory>: View {
                     message: message,
                     contentWidth: contentWidth,
                     isFirst: showsAllInfo,
-                    scrolledId: $scrolledId
+                    scrolledId: $scrolledId,
+                    translationLanguage: messageViewModel.translationLanguage
                 )
             }
         }

Sources/StreamChatSwiftUI/ChatMessageList/MessageItemView.swift

@@ -144,8 +144,6 @@ public struct MessageItemView<Factory: ViewFactory>: View {
         )
         .accessibilityElement(children: .contain)
         .accessibilityIdentifier("MessageItemView")
-        // TODO: Refactor so LinkDetectionTextView does not depend directly on the view model through @Environment.
-        .environment(\.messageViewModel, messageViewModel)
         .onChange(of: message) { message in messageViewModel.message = message }
         .onChange(of: channel) { channel in messageViewModel.channel = channel }
     }

Sources/StreamChatSwiftUI/ChatMessageList/MessageListView.swift

@@ -206,7 +206,6 @@ public struct MessageListView<Factory: ViewFactory>: View, KeyboardReadable {
                                     isLast: !showsLastInGroupInfo && message == messages.last
                                 )
                             )
-                            .environment(\.channelTranslationLanguage, channel.membership?.language)
                             .onAppear {
                                 if index == nil {
                                     index = messageListDateUtils.index(for: message, in: messages)
@@ -720,31 +719,3 @@ private class MessageRenderingUtil {
         return skipRendering
     }
 }
-
-private struct ChannelTranslationLanguageKey: EnvironmentKey {
-    static let defaultValue: TranslationLanguage? = nil
-}
-
-private struct MessageViewModelKey: EnvironmentKey {
-    static let defaultValue: MessageViewModel? = nil
-}
-
-extension EnvironmentValues {
-    var channelTranslationLanguage: TranslationLanguage? {
-        get {
-            self[ChannelTranslationLanguageKey.self]
-        }
-        set {
-            self[ChannelTranslationLanguageKey.self] = newValue
-        }
-    }
-    
-    var messageViewModel: MessageViewModel? {
-        get {
-            self[MessageViewModelKey.self]
-        }
-        set {
-            self[MessageViewModelKey.self] = newValue
-        }
-    }
-}

Sources/StreamChatSwiftUI/ChatMessageList/MessageView.swift

@@ -16,19 +16,22 @@ public struct MessageView<Factory: ViewFactory>: View {
     public var message: ChatMessage
     public var contentWidth: CGFloat
     public var isFirst: Bool
+    public var translationLanguage: TranslationLanguage?
     @Binding public var scrolledId: String?
 
     public init(
         factory: Factory,
         message: ChatMessage,
         contentWidth: CGFloat,
         isFirst: Bool,
-        scrolledId: Binding<String?>
+        scrolledId: Binding<String?>,
+        translationLanguage: TranslationLanguage? = nil
     ) {
         self.factory = factory
         self.message = message
         self.contentWidth = contentWidth
         self.isFirst = isFirst
+        self.translationLanguage = translationLanguage
         _scrolledId = scrolledId
     }
 
@@ -75,7 +78,8 @@ public struct MessageView<Factory: ViewFactory>: View {
                         message: message,
                         isFirst: isFirst,
                         availableWidth: contentWidth,
-                        scrolledId: $scrolledId
+                        scrolledId: $scrolledId,
+                        translationLanguage: translationLanguage
                     )
                 )
             } else {
@@ -93,7 +97,8 @@ public struct MessageView<Factory: ViewFactory>: View {
                             message: message,
                             isFirst: isFirst,
                             availableWidth: contentWidth,
-                            scrolledId: $scrolledId
+                            scrolledId: $scrolledId,
+                            translationLanguage: translationLanguage
                         )
                     )
                 }
@@ -110,6 +115,7 @@ public struct MessageTextView<Factory: ViewFactory>: View {
     private let factory: Factory
     private let message: ChatMessage
     private let isFirst: Bool
+    private let translationLanguage: TranslationLanguage?
     private let leadingPadding: CGFloat
     private let trailingPadding: CGFloat
     private let topPadding: CGFloat
@@ -120,7 +126,8 @@ public struct MessageTextView<Factory: ViewFactory>: View {
         factory: Factory,
         message: ChatMessage,
         isFirst: Bool,
-        scrolledId: Binding<String?>
+        scrolledId: Binding<String?>,
+        translationLanguage: TranslationLanguage?
     ) {
         @Injected(\.tokens) var tokens
         self.init(
@@ -131,10 +138,11 @@ public struct MessageTextView<Factory: ViewFactory>: View {
             trailingPadding: tokens.spacingSm,
             topPadding: tokens.spacingXs,
             bottomPadding: tokens.spacingXs,
-            scrolledId: scrolledId
+            scrolledId: scrolledId,
+            translationLanguage: translationLanguage
         )
     }
-    
+
     public init(
         factory: Factory,
         message: ChatMessage,
@@ -143,11 +151,13 @@ public struct MessageTextView<Factory: ViewFactory>: View {
         trailingPadding: CGFloat,
         topPadding: CGFloat,
         bottomPadding: CGFloat,
-        scrolledId: Binding<String?>
+        scrolledId: Binding<String?>,
+        translationLanguage: TranslationLanguage?
     ) {
         self.factory = factory
         self.message = message
         self.isFirst = isFirst
+        self.translationLanguage = translationLanguage
         self.leadingPadding = leadingPadding
         self.trailingPadding = trailingPadding
         self.topPadding = topPadding
@@ -160,12 +170,15 @@ public struct MessageTextView<Factory: ViewFactory>: View {
             alignment: message.alignmentInBubble,
             spacing: 0
         ) {
-            factory.makeStreamTextView(options: .init(message: message))
-                .padding(.leading, leadingPadding)
-                .padding(.trailing, trailingPadding)
-                .padding(.top, topPadding)
-                .padding(.bottom, bottomPadding)
-                .fixedSize(horizontal: false, vertical: true)
+            factory.makeStreamTextView(options: .init(
+                message: message,
+                translationLanguage: translationLanguage
+            ))
+            .padding(.leading, leadingPadding)
+            .padding(.trailing, trailingPadding)
+            .padding(.top, topPadding)
+            .padding(.bottom, bottomPadding)
+            .fixedSize(horizontal: false, vertical: true)
         }
         .modifier(
             factory.styles.makeMessageViewModifier(
@@ -220,117 +233,34 @@ public struct EmojiTextView<Factory: ViewFactory>: View {
 }
 
 struct StreamTextView: View {
-    @Injected(\.fonts) var fonts
-    
-    let message: ChatMessage
-    private let adjustedText: String
-    
-    init(message: ChatMessage) {
-        self.message = message
-        adjustedText = message.adjustedText
-    }
-    
-    var body: some View {
-        if #available(iOS 15, *) {
-            LinkDetectionTextView(message: message)
-        } else {
-            Text(adjustedText)
-                .foregroundColor(textColor(for: message))
-                .font(fonts.body)
-        }
-    }
-}
-
-@available(iOS 15, *)
-public struct LinkDetectionTextView: View {
     @Environment(\.layoutDirection) var layoutDirection
-    @Environment(\.channelTranslationLanguage) var translationLanguage
-    @Environment(\.messageViewModel) var messageViewModel
-
     @Injected(\.colors) var colors
     @Injected(\.fonts) var fonts
-    @Injected(\.utils) var utils
-    
-    var message: ChatMessage
 
-    // The translations store is used to detect changes so the textContent is re-rendered.
-    // The @Environment(\.messageViewModel) is not reactive like @EnvironmentObject.
-    // TODO: On v5 the TextView should be refactored and not depend directly on the view model.
-    @ObservedObject var originalTranslationsStore = InjectedValues[\.utils].originalTranslationsStore
+    let message: ChatMessage
+    let textContent: String
+    let translationLanguage: TranslationLanguage?
 
-    @State var text: AttributedString?
-    @State var linkDetector = TextLinkDetector()
-    @State var tintColor = Color(InjectedValues[\.colors].accentPrimary)
-        
-    public init(
-        message: ChatMessage
-    ) {
+    init(message: ChatMessage, translationLanguage: TranslationLanguage?) {
         self.message = message
+        self.textContent = message.textContent(for: translationLanguage) ?? message.adjustedText
+        self.translationLanguage = translationLanguage
     }
-    
-    public var body: some View {
-        Group {
-            Text(text ?? displayText)
-        }
-        .foregroundColor(textColor(for: message))
-        .font(fonts.body)
-        .tint(tintColor)
-        .onChange(of: message) { message in
-            messageViewModel?.message = message
-            text = displayText
-        }
-    }
-    
-    var displayText: AttributedString {
-        let text = messageViewModel?.textContent ?? message.text
 
-        // Markdown
-        let attributes = AttributeContainer()
-            .foregroundColor(textColor(for: message))
-            .font(fonts.body)
-        var attributedString: AttributedString
-        if utils.messageListConfig.markdownSupportEnabled {
-            attributedString = utils.markdownFormatter.format(
-                text,
-                attributes: attributes,
-                layoutDirection: layoutDirection
+    var body: some View {
+        if #available(iOS 15, *) {
+            let attributedText = message.attributedTextContent(
+                layoutDirection: layoutDirection,
+                translationLanguage: translationLanguage
             )
+            Text(attributedText)
+                .foregroundColor(textColor(for: message))
+                .font(fonts.body)
+                .tint(Color(colors.accentPrimary))
         } else {
-            attributedString = AttributedString(message.adjustedText, attributes: attributes)
-        }
-        // Links and mentions
-        if utils.messageListConfig.localLinkDetectionEnabled {
-            for user in message.mentionedUsers {
-                let mention = "@\(user.name ?? user.id)"
-                let ranges = attributedString.ranges(of: mention, options: [.caseInsensitive])
-                for range in ranges {
-                    if let messageId = message.messageId.addingPercentEncoding(withAllowedCharacters: .urlPathAllowed),
-                       let url = URL(string: "getstream://mention/\(messageId)/\(user.id)") {
-                        attributedString[range].link = url
-                    }
-                }
-            }
-            for link in linkDetector.links(in: String(attributedString.characters)) {
-                if let attributedStringRange = Range(link.range, in: attributedString) {
-                    attributedString[attributedStringRange].link = link.url
-                }
-            }
-        }
-        // Finally change attributes for links (markdown links, text links, mentions)
-        var linkAttributes = utils.messageListConfig.messageDisplayOptions.messageLinkDisplayResolver(message)
-        if !linkAttributes.isEmpty {
-            var linkAttributeContainer = AttributeContainer()
-            if let uiColor = linkAttributes[.foregroundColor] as? UIColor {
-                linkAttributeContainer = linkAttributeContainer.foregroundColor(Color(uiColor: uiColor))
-                linkAttributes.removeValue(forKey: .foregroundColor)
-            }
-            linkAttributeContainer.merge(AttributeContainer(linkAttributes))
-            for (value, range) in attributedString.runs[\.link] {
-                guard value != nil else { continue }
-                attributedString[range].mergeAttributes(linkAttributeContainer)
-            }
+            Text(textContent)
+                .foregroundColor(textColor(for: message))
+                .font(fonts.body)
         }
-        
-        return attributedString
     }
 }