Implement improved rendering for threads in the main timeline

[janogarcia]

Changelog

Root messages should form a standalone group with their respective thread summaries.

This way, we establish a clear relationship between the thread summary and the root message it belongs to. Failure to do it results in an ambiguous relationship that only gets worse when message bubbles are enabled.

Source

→ https://www.figma.com/file/T309ztx0sNyOOK6NKVLHsK/Threads?node-id=3085%3A306074

Snapshot

[janogarcia]

I've updated the issue description and mockup for better clarity on what the expected rendering should be.

@novocaine This issue is currently assigned to Germain. I guess we should either unassign or reassign it. @robintown took the lead on implementing the current behavior, which is already quite close to the expected one.

[t3chguy]

@janogarcia I am failing to see the difference between the Figma and existing implementation

Could you please point out what you need changing?

Thanks

[janogarcia]

@t3chguy Sure, in the example below the threads (third messages in the group) shouldn't be grouped with the messages first and second. That is, make sure the threads are always rendered in standalone groups. Does that make sense?

[t3chguy]

Oh so don't form a continuation on both sides of the thread preview tile, got it.

Though the Figma does not happen to show that ftr

[janogarcia]

If the first message were a regular message, it would be rendered the same. I should have made that more explicit in the Figma mockup.