Skip to content

Conversation

@JcMinarro
Copy link
Contributor

@JcMinarro JcMinarro commented May 8, 2023

🎯 Goal

Be able to set position where the thread messages will start from.
Complete GetStream/android-internal-board#62

🎨 UI Changes

Add relevant screenshots

Bottom Top
Screenshot_20230509_112730 Screenshot_20230509_113943

🎉 GIF

@JcMinarro JcMinarro force-pushed the thread-messages-start-feature branch from 4daf5d7 to c037254 Compare May 8, 2023 16:47
@JcMinarro JcMinarro marked this pull request as ready for review May 9, 2023 09:44
@JcMinarro JcMinarro requested a review from a team as a code owner May 9, 2023 09:44
@JcMinarro JcMinarro requested a review from kanat May 9, 2023 09:44
Copy link
Contributor

@kanat kanat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Are we gonna support this in XML as well?

@JcMinarro
Copy link
Contributor Author

LGTM.

Are we gonna support this in XML as well?

It was already implemented on XML: https://getstream.io/chat/docs/sdk/android/ui/message-components/message-list/#messages-start-position

@JcMinarro JcMinarro merged commit 656d801 into v5 May 10, 2023
@JcMinarro JcMinarro deleted the thread-messages-start-feature branch May 10, 2023 08:18
@ChrisTitos
Copy link

After updating the SDK to the latest version, suddenly the messages in channels (not threads) are always aligned at the top.
I tracked it down to this pull request, because I think this changed the old behavior (which was align bottom)

You see on this line if there is no parent message, alignment is Top, but it used to be Bottom before this change.
There is also no option provided to change this behavior.

Should I make a new bug ticket for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants