Fix issues around read receipt

[bmarty]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

This PR fixes several issues around read receipt:

• make the RR list scrollable in the bottom sheet of RR
• render read receipt for state events
• render aggregated read receipt for grouped state events. Note that the RRs are not clickable here, the group has to be expanded first.
• reduce bottom padding of state event (more compact timeline and also less space if there are some RR)
• add preview for expanded group of state event

Motivation and context

Reliable read receipt feature

Screenshots / GIFs

Tests

• Enable RR rendered in advanced settings.
• Open a room with many people, and observe that Read Receipt are rendered correctly. Note: there is still an issue on Rust side: Invisible Read Receipt matrix-org/matrix-rust-sdk#2889

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 23
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/vector-im/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request includes a new file under ./changelog.d. See https://github.com/vector-im/element-x-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/ovp5Tn

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[codecov]

Codecov Report

Attention: 41 lines in your changes are missing coverage. Please review.

Comparison is base (4de256b) 66.48% compared to head (bc60abd) 66.57%.
Report is 4 commits behind head on develop.

Files	Patch %	Lines
...imeline/components/TimelineItemGroupedEventsRow.kt	80.82%	8 Missing and 6 partials ⚠️
...ents/receipt/bottomsheet/ReadReceiptBottomSheet.kt	27.77%	10 Missing and 3 partials ⚠️
...ssages/impl/timeline/components/TimelineItemRow.kt	81.35%	7 Missing and 4 partials ⚠️
...l/timeline/components/TimelineItemStateEventRow.kt	90.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #1910      +/-   ##
===========================================
+ Coverage    66.48%   66.57%   +0.08%     
===========================================
  Files         1308     1310       +2     
  Lines        33058    33138      +80     
  Branches      7084     7097      +13     
===========================================
+ Hits         21979    22061      +82     
+ Misses        7724     7717       -7     
- Partials      3355     3360       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[julioromano]

LGTM
Just a question, in: 40c91a8
Where is the actual background worker?

[bmarty]

LGTM Just a question, in: 40c91a8 Where is the actual background worker?

TimelineItemGrouper.kt is used by the Factory