Custom subview of ChatMessageContentView is not sized correctly about stream-chat-swift HOT 5 CLOSED

Hi @srgray,

Yes, that is normal, unfortunately. Since our Message List is an inverted UITableView and not a UICollectionView, embedding self-resizing collection views won't work very well. Although it might be possible to build that UI with some workarounds. You can try the following:

1. If possible, use a UIStackView. If you are not expecting many tags per message, this could be enough, and the performance penalty of using a stack view won't be noticeable.
2. The other option is to find a way to explicitly reload the cell whenever the tags grow to a specific size. One idea is to override the ChatMessageListView.reuseIdentifier() function which is the one responsible for calculating the cell identifier based on the ChatMessageLayoutOption and the message content. You could use the tags.countand post-fix the cell identifier to make sure that whenever the tags change, it reloads the cell 🤔

The pseudo code for 2:

class CustomMessageListView: ChatMessageListView {
    override func reuseIdentifier(
        contentViewClass: ChatMessageContentView.Type,
        attachmentViewInjectorType: AttachmentViewInjector.Type?,
        layoutOptions: ChatMessageLayoutOptions,
        message: ChatMessage?
    ) -> String {
        var identifier = super.reuseIdentifier(
            contentViewClass: contentViewClass,
            attachmentViewInjectorType: attachmentViewInjectorType,
            layoutOptions: layoutOptions,
            message: message
        )
        if message.tags.count > 0 {
            identifier += "_\(message.tags.count)"
        }
        return identifier
    }
}

Let us know if you were able to find a workaround for this.

Best,
Nuno

from stream-chat-swift.

@srgray We have a similar UI in our Slack Example App. We use a collection view there, but we have a height constraint that we change depending on the number of reactions.

The code:

• https://github.com/GetStream/stream-chat-swift/blob/develop/Examples/SlackClone/SlackChatMessageContentView.swift
• https://github.com/GetStream/stream-chat-swift/blob/develop/Examples/SlackClone/SlackReactionsView.swift

This works best whenever the collection view has a predictable number of items per row since we can do this to calculate the height constraint:

let numberOfItemsPerRow = 3
let numberOfRows = Double(reactions.count).truncatingRemainder(dividingBy: numberOfItemsPerRow)
heightConstraint?.constant = ceil(numberOfRows) * reactionRowHeight

But yeah, these use cases are a bit tricky when working with a UITableView, unfortunatly it is a short-coming of our Message List at the moment, and solving it completely would require refactoring it to a UICollectionView, which we are not planning to do at the moment.

Best,
Nuno

from stream-chat-swift.

Hi @srgray,

Did one of the solutions provided work for you? If yes, which option did you choose?

Best,
Nuno

from stream-chat-swift.

Hey @srgray,

I'm closing this for now. Feel free to reply and re open it in case you are still having problems or want to share your solution.

Best,
Nuno

from stream-chat-swift.